Skip to content

Fix CI#18168

Merged
charlesBochet merged 1 commit intomainfrom
ci-fix
Feb 23, 2026
Merged

Fix CI#18168
charlesBochet merged 1 commit intomainfrom
ci-fix

Conversation

@charlesBochet
Copy link
Copy Markdown
Member

@charlesBochet charlesBochet commented Feb 23, 2026

Fix 2 issues on CI:

  • website still using ubuntu-latest
  • linter that cannot use more than 2GB

Copy link
Copy Markdown
Contributor

@bosiraphael bosiraphael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@charlesBochet charlesBochet merged commit 7944ca2 into main Feb 23, 2026
50 of 58 checks passed
@charlesBochet charlesBochet deleted the ci-fix branch February 23, 2026 11:05
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Feb 23, 2026

Greptile Summary

This PR addresses CI infrastructure issues by standardizing runner configurations and increasing memory allocation for Node.js processes.

Changes made:

  • Added NODE_OPTIONS: '--max-old-space-size=4096' environment variable to the front-task job in ci-front.yaml to provide 4GB of heap memory for frontend tasks (lint, typecheck, test)
  • Migrated ci-website.yaml jobs from ubuntu-latest to depot-ubuntu-24.04 runners for consistency with other CI jobs in the workflow

These changes likely address out-of-memory errors or performance issues in the CI pipeline.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are configuration-only modifications to CI workflow files that increase memory allocation and standardize runner infrastructure. These are common fixes for CI stability issues and do not affect application code, logic, or user-facing functionality.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/ci-front.yaml Added NODE_OPTIONS: '--max-old-space-size=4096' to front-task job to increase memory allocation for frontend tasks
.github/workflows/ci-website.yaml Changed runner from ubuntu-latest to depot-ubuntu-24.04 for website-build and ci-website-status-check jobs

Last reviewed commit: c7c7ea9

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@twenty-eng-sync
Copy link
Copy Markdown

Hey @charlesBochet! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants