fix(ci): Increase build application memory#4140
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit 4eec739. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR bumps the Node.js heap limit to 8 GB ( Confidence Score: 5/5Safe to merge — minimal, targeted CI fix with no risk of regression. Single-line change in two places, both well-scoped to the affected CI steps. The 8 GB limit is appropriate for an 8 vCPU / 16 GB RAM Blacksmith runner. No logic, application code, or tests are affected. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Checkout & Setup] --> B[Install Dependencies]
B --> C[Validate Feature Flags]
C --> D[Check Subblock ID Stability]
D --> E[Lint Code]
E --> F["Run Tests with Coverage\nNODE_OPTIONS: --no-warnings --max-old-space-size=8192"]
F --> G[Check Schema & Migrations]
G --> H["Build Application\nNODE_OPTIONS: --no-warnings --max-old-space-size=8192"]
H --> I[Upload Coverage to Codecov]
style F fill:#f9f,stroke:#333
style H fill:#f9f,stroke:#333
Reviews (1): Last reviewed commit: "Increase build application memory" | Re-trigger Greptile |
Summary
We added some more dependencies causing build to fail. Increased memory to unblock build in ci.
Type of Change
Testing
Checklist
Screenshots/Videos