Skip to content

chore: upgrade GitHub Actions runtime from node20 to node24#33

Merged
ryechezkel merged 7 commits intomainfrom
chore/upgrade-node-24
Apr 16, 2026
Merged

chore: upgrade GitHub Actions runtime from node20 to node24#33
ryechezkel merged 7 commits intomainfrom
chore/upgrade-node-24

Conversation

@ryechezkel
Copy link
Copy Markdown
Contributor

@ryechezkel ryechezkel commented Apr 15, 2026

User description

Summary

  • Upgrades runs.using in action.yml from node20 to node24 ahead of GitHub's node20 deprecation
  • Updates the README prerequisites to reflect the new runtime version

Generated description

Below is a concise technical summary of the changes proposed in this PR:
Raise the GitHub Action runtime to Node 24 across the action metadata, release workflow, and package configuration so the compiled code and engines declare version 3.0.0. Clarify in the README that the prerequisite runtime is Node 24 and that users should reference the v3 action.

TopicDetails
Docs update Refresh the README prerequisites and usage snippets to reflect the Node 24 runtime and the v3 action tag.
Modified files (1)
  • README.md
Latest Contributors(2)
UserCommitDate
ryechezkelfeat: post PR comment ...March 11, 2026
rabinovich@groundcover...Document test result s...March 08, 2026
Runtime upgrade Raise the action runtime to Node 24 by updating runs.using along with the release workflow node version and package metadata so dist/index.js now ships as version 3.0.0.
Modified files (4)
  • .github/workflows/release.yml
  • action.yml
  • dist/index.js
  • package.json
Latest Contributors(2)
UserCommitDate
ryechezkelchore: fix npm audit v...April 16, 2026
rabinovich@groundcover...chore: bump version to...March 12, 2026
This pull request is reviewed by Baz. Review like a pro on (Baz).

Summary by CodeRabbit

  • New Features

    • Major version release (v3.0.0) with updated runtime compatibility.
  • Documentation

    • Updated GitHub Actions integration guidance and workflow examples to reflect latest action version.
  • Chores

    • Updated minimum Node.js requirement from version 20 to version 24 across the GitHub Action runtime and package dependencies.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f21d4e03-e9f4-4467-affa-bb7fe19373e2

📥 Commits

Reviewing files that changed from the base of the PR and between c313399 and 2e05e11.

📒 Files selected for processing (1)
  • README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • README.md

Walkthrough

Node.js runtime requirement raised from 20 → 24 in the action metadata and CI; package version bumped to 3.0.0 and engines.node updated; README workflow examples updated to use groundcover-com/groundcover-github-action@v3.

Changes

Cohort / File(s) Summary
README workflow examples
README.md
All workflow snippets updated to use groundcover-com/groundcover-github-action@v3; noted self-hosted runner and runner version requirements.
Action runtime declaration
action.yml
Changed action runtime runs.using from node20node24 (entry dist/index.js unchanged).
Package manifest
package.json
Bumped version 2.4.03.0.0; raised engines.node >=20.0.0>=24.0.0.
CI workflow
.github/workflows/release.yml
Updated release workflow Node setup from Node 20 → Node 24 via actions/setup-node@v4.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive Description covers the changes (runtime upgrade, README updates) and mentions the version bump to 3.0.0, but does not follow the provided template structure with Type of Change, Testing, or Checklist sections. Consider using the repository's PR template: include Type of Change (e.g., 'Breaking change' for major version), Testing approach, and completion of the Checklist items.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed Title directly summarizes the main change: upgrading GitHub Actions runtime from Node 20 to Node 24. It is concise, specific, and reflects the primary purpose of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/upgrade-node-24

Comment @coderabbitai help to get the list of available commands and usage tips.

@ryechezkel ryechezkel requested a review from maxlevin-gc April 15, 2026 15:41
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 15, 2026

Open in groundcover

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
action.yml (1)

95-95: Update package.json engines.node to match action.yml runtime.

Line 95 specifies Node 24, but package.json (line 86) declares >=20.0.0. Bump to >=24.0.0 or document if lower versions are still supported locally to prevent contributor/CI version drift.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@action.yml` at line 95, The package.json engines.node value must match the
action.yml runtime; update the package.json "engines": { "node": ... } entry
(currently ">=20.0.0") to ">=24.0.0" (or explicitly document supported local
versions) so it aligns with action.yml's using: "node24"; locate the
engines.node field in package.json and bump it to >=24.0.0 (or add a comment in
README if you intentionally support older local versions).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@action.yml`:
- Line 95: The package.json engines.node value must match the action.yml
runtime; update the package.json "engines": { "node": ... } entry (currently
">=20.0.0") to ">=24.0.0" (or explicitly document supported local versions) so
it aligns with action.yml's using: "node24"; locate the engines.node field in
package.json and bump it to >=24.0.0 (or add a comment in README if you
intentionally support older local versions).

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4f84f418-9255-4f84-b979-f0f37f3f89bb

📥 Commits

Reviewing files that changed from the base of the PR and between e937b24 and c765568.

📒 Files selected for processing (2)
  • README.md
  • action.yml

Comment thread action.yml
@ryechezkel ryechezkel requested review from yossilevi-web and removed request for maxlevin-gc April 16, 2026 05:47
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Around line 28-31: Update the invalid action ref
"groundcover-com/groundcover-github-action@v3" used in README examples to a
valid ref: either create and push a "v3" tag in the action repo or replace every
occurrence of "groundcover-com/groundcover-github-action@v3" (seen throughout
the README examples) with an existing tag, a commit SHA, or a stable branch ref
(e.g., `@main`); ensure all listed example usages are changed consistently so the
workflows run without failing.
- Line 8: Replace the misleading README line "Node 24 runtime (handled
automatically by GitHub Actions)" with a clear note that Node 24 requires
explicit runner support: state that you must set runs.using: node24 in
action.yml and that GitHub Actions runners require version v2.327.1+ (both
GitHub-hosted and self-hosted) because the default runner uses Node 20 until
June 2, 2026; update the README.md text to include these exact prerequisites and
mention self-hosted runners with older runner versions will fail.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b12796b4-0d60-4d41-9a1c-eb16f8780768

📥 Commits

Reviewing files that changed from the base of the PR and between 85d4611 and c313399.

⛔ Files ignored due to path filters (1)
  • dist/index.js is excluded by !**/dist/**
📒 Files selected for processing (3)
  • .github/workflows/release.yml
  • README.md
  • package.json
✅ Files skipped from review due to trivial changes (1)
  • .github/workflows/release.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • package.json

Comment thread README.md Outdated
Comment thread README.md
@ryechezkel ryechezkel merged commit e8d7968 into main Apr 16, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants