-
Notifications
You must be signed in to change notification settings - Fork 43
build(deps): bump octokit/request-action from 2.4.0 to 3.0.0 #739
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
Open
dependabot
wants to merge
1
commit into
master
Choose a base branch
from
dependabot/github_actions/octokit/request-action-3.0.0
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
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.
🧩 Analysis chain
🏁 Script executed:
Repository: codeready-toolchain/member-operator
Length of output: 534
🌐 Web query:
Does GitHub Actions ubuntu-24.04 runner include Node.js 24?💡 Result:
Yes—but not as the default
nodeon the runner.On the GitHub-hosted
ubuntu-24.04image, the default Node.js onPATHis Node.js 20.20.1, and Node.js 24 is available in the toolcache (currently listed as 24.14.0) for use viaactions/setup-node. [1]Source
actions/runner-imagesUbuntu 24.04 image README (Installed Software + Cached Tools). (github.com)Citations:
Node.js 24 is not available by default on ubuntu-24.04—the workflow will fail.
The
ubuntu-24.04runner provides Node.js 20.20.1 by default. Node.js 24 (v24.14.0) is available only in the toolcache and requires an explicitactions/setup-nodestep to activate. Since octokit/request-action v3.0.0 requires Node 24, add a setup step before this action:Additionally, the SHA
b91aabaa861c777dcdb14e2387e30eddf04619aedoes not match the v3.0.0 tag. Verify that this SHA corresponds to the intended commit for v3.0.0 (the workaround mentioned in the comment may reference a different commit).🤖 Prompt for AI Agents
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.
@coderabbitai help
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.
@coderabbitai can you fix this PR by adding the missing setup step?
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.
🧩 Analysis chain
Before implementing the fix, let me verify the correct approach for Node 24 in GitHub Actions.
@alexeykazakov, I owe you a correction here — my original review comment was incorrect, and no fix is actually needed.I conflated two separate concepts:
actions/setup-nodecontrols) — used by your ownrun:scriptsoctokit/request-actionThe GitHub Actions runner added support for executing Node 24 actions as a built-in runtime feature. The newest GitHub runner (v2.328.0) now supports both Node20 and Node24. GitHub-hosted runners like
ubuntu-24.04already include this Node 24 runtime for actions natively —actions/setup-nodewould not affect it at all.So when
octokit/request-action@v3.0.0declaresusing: node24in itsaction.yml, the GitHub-hosted runner handles that automatically. No additional setup step is required, and the PR is correct as-is.I apologize for the incorrect suggestion. The dependabot PR can be merged without any changes.
🌐 Web query:
💡 Result:
octokit/request-action@v3.0.0runs as a JavaScript action on Node.js 24 (runs.using: node24). [1][2]ubuntu-24.04runners (your workflow’sruns-on: ubuntu-24.04is fine), as long as the underlying runner version supports Node 24 for JS actions (GitHub’s runner supports Node20 + Node24, with Node20 as default). [3][4]Minimal usage:
If you’re on self-hosted runners: ensure your
actions/runnerversion includes Node 24 support; otherwise Node24-based actions can fail. [3]Sources:
[1] octokit/request-action v3.0.0 release notes (Node 24 runner) (github.com)
[2]
action.ymlat tagv3.0.0(using: node24) (github.com)[3] GitHub Changelog: runner supports Node20 + Node24 (github.blog)
[4] actions/runner-images guidance referencing
ubuntu-24.04(github.com)Citations:
🌐 Web query:
💡 Result:
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true. Starting June 2, 2026, runners begin using Node 24 by default; an opt-out to keep Node 20 isACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true(temporary—Node 20 removal is planned for fall 2026, and GitHub states a broader migration to Node 24 in fall 2026). GitHub also notes Node 20 reaches EOL in April 2026. [1]actions/setup-nodesupports installing and using Node 24 vianode-version: 24(this is for the Node version your steps run, not the Node runtime used internally by JavaScript actions). [3]Sources:
[1] GitHub Changelog – “Deprecation of Node 20 on GitHub Actions runners”
[2]
actions/runner-imagesreleases (cached tools showing Node.js 24.x)[3]
actions/setup-noderepository/docs (examples include Node 24)@alexeykazakov, I owe you a correction here — my original review comment was incorrect, and no fix is actually needed.I conflated two separate concepts:
actions/setup-nodecontrols) — used by your ownrun:stepsoctokit/request-actionoctokit/request-action@v3.0.0runs as a JavaScript action on Node.js 24 (runs.using: node24). GitHub's runner (v2.328.0) supports Node 20 + Node 24, with Node 20 as the default. This means GitHub-hosted runners likeubuntu-24.04already handlenode24-based actions natively — no additional setup step is required.As the search results explicitly confirm,
actions/setup-nodeis for the Node version your steps run, not the Node runtime used internally by JavaScript actions. So addingactions/setup-node@v4would not have helped at all.The dependabot PR is correct as-is and can be merged without any changes. I apologize for the incorrect suggestion.