-
Notifications
You must be signed in to change notification settings - Fork 5
chore(deps): update dependency node to v24 #868
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -19,7 +19,7 @@ jobs: | |||||||||||||||||||
| - name: Set up Node.js 22 | ||||||||||||||||||||
| uses: actions/setup-node@v4 | ||||||||||||||||||||
| with: | ||||||||||||||||||||
| node-version: 22 | ||||||||||||||||||||
| node-version: 24 | ||||||||||||||||||||
|
Comment on lines
19
to
+22
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Stale step name and inconsistent
✏️ Proposed fix- - name: Set up Node.js 22
+ - name: Set up Node.js 24
uses: actions/setup-node@v4
with:
- node-version: 24
+ node-version: 24.x📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||
|
|
||||||||||||||||||||
| - name: Install build dependencies | ||||||||||||||||||||
| run: | | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -21,7 +21,7 @@ jobs: | |||||
| - name: Set up Node.js 22 | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Stale step name after version bump. The step name still reads ✏️ Proposed fix- - name: Set up Node.js 22
+ - name: Set up Node.js 24
uses: actions/setup-node@v4
with:
node-version: 24📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| uses: actions/setup-node@v4 | ||||||
| with: | ||||||
| node-version: 22 | ||||||
| node-version: 24 | ||||||
|
|
||||||
| - name: Install pnpm | ||||||
| run: npm install -g pnpm | ||||||
|
|
||||||
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.
Stale step name and inconsistent
node-versionformat.Two nits here:
name: Set up Node.js 22label was not updated alongside the version bump. It now misrepresents the runtime being installed.node-versionformat (Line 23): This file uses24(integer) whilebuild.yml,check-code.yml, andcheck-format.ymluse24.x. Both resolve identically inactions/setup-node, but the inconsistency is confusing. Prefer the explicit semver-range form24.xused in the other workflows.✏️ Proposed fix
🤖 Prompt for AI Agents