-
Notifications
You must be signed in to change notification settings - Fork 60
Upgrade Node to v24 (LTS) #1225
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
🦋 Changeset detectedLatest commit: 5ccfba3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 8 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
Pull request overview
This PR upgrades Node.js from v22 to v24 across the repository, including package dependencies, GitHub Actions workflows, and development container configurations.
Changes:
- Updated @types/node dependency from 22.18.1 to 24.10.9
- Updated minimum Node.js engine requirement from >=22.0.0 to >=24.0.0
- Updated node-version in all GitHub Actions workflows from 22 to 24
- Updated devcontainer image from typescript-node:22 to typescript-node:24
Reviewed changes
Copilot reviewed 16 out of 17 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated @types/node to v24.10.9 and engines.node to >=24.0.0 |
| .github/workflows/ci.yml | Updated node-version to 24 for CI workflow |
| .github/workflows/deploy_docs.yml | Updated node-version to 24 for docs deployment |
| .github/workflows/deploy_docs_preview.yml | Updated node-version to 24 for preview deployments |
| .github/workflows/deploy_docs_preview_forks.yml | Updated node-version to 24 for fork preview deployments |
| .github/workflows/figma_connect_publish.yml | Updated node-version to 24 for Figma Connect publishing |
| .github/workflows/integration_test_astro.yml | Updated node-version to 24 for Astro integration tests |
| .github/workflows/integration_test_nextjs.yml | Updated node-version to 24 for Next.js integration tests |
| .github/workflows/integration_test_remix.yml | Updated node-version to 24 for Remix integration tests |
| .github/workflows/release_candidate_checklist.yml | Updated node-version to 24 for release candidate workflow |
| .github/workflows/ui_test_accessibility.yml | Updated node-version to 24 for accessibility tests |
| .github/workflows/ui_test_diff_comment.yml | Updated node-version to 24 for UI diff commenting |
| .github/workflows/ui_test_primitives_diff.yml | Updated node-version to 24 for primitives diff tests |
| .github/workflows/update_visual_snapshots.yml | Updated node-version to 24 for visual snapshot updates |
| .github/workflows/visual_test_storybook.yml | Updated node-version to 24 for Storybook visual tests |
| .devcontainer/devcontainer.json | Updated devcontainer image to typescript-node:24 |
🟢 No design token changes found |
🟢 No unit test coverage changes foundAll components and hooks with tests maintain the same coverage as the main branch. |
🟢 No visual differences foundOur visual comparison tests did not find any differences in the UI. |
danielguillan
left a comment
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.
👍
Summary
Updates Node.js to v24 across the repo, which was recently give LTS status.
Contributor checklist:
update snapshotslabel to the PR)Reviewer checklist:
Screenshots: