-
Notifications
You must be signed in to change notification settings - Fork 15
fix(netlify): Fix Netlify deploy to use Node 24 #946
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Deploy Preview for cedarjs canceled.
|
Greptile SummaryUpdated the Netlify deployment template to use Node 24, aligning with Cedar's current Node version requirements (project templates already specify
The change is necessary because current Cedar versions no longer work with Node 20. Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Dev as Developer
participant CLI as Cedar CLI
participant Template as netlify.js Template
participant Netlify as Netlify Platform
Dev->>CLI: Run setup deploy command
CLI->>Template: Load Netlify config template
Template->>Template: Generate netlify.toml with Node 24
Template->>CLI: Return config string
CLI->>Dev: Create netlify.toml file
Dev->>Netlify: Deploy with netlify.toml
Netlify->>Netlify: Use NODE_VERSION=24 from config
Netlify->>Netlify: Build with yarn cedar deploy netlify
Netlify->>Dev: Deployment complete
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 file reviewed, 1 comment
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx run-many -t build:pack --exclude create-ceda... |
✅ Succeeded | 6s | View ↗ |
nx run-many -t build |
✅ Succeeded | 6s | View ↗ |
nx run-many -t test --minWorkers=1 --maxWorkers=4 |
✅ Succeeded | 1m 32s | View ↗ |
nx run-many -t test:types |
✅ Succeeded | 9s | View ↗ |
☁️ Nx Cloud last updated this comment at 2026-01-07 08:55:48 UTC
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx run-many -t test --minWorkers=1 --maxWorkers=4 |
❌ Failed | 1m 13s | View ↗ |
nx run-many -t build:pack --exclude create-ceda... |
✅ Succeeded | 5s | View ↗ |
nx run-many -t build |
✅ Succeeded | 3m 26s | View ↗ |
nx run-many -t test:types |
✅ Succeeded | 6s | View ↗ |
☁️ Nx Cloud last updated this comment at 2026-01-06 23:49:21 UTC

Updating the template for
netlify.tomlwith the most important change being the upgrade to Node 24. Current Cedar versions no longer work with Node 20