Skip to content

👷 Force Node 24 in CI#19

Merged
HanXHX merged 1 commit intomasterfrom
fix_ci_deprecations
Mar 12, 2026
Merged

👷 Force Node 24 in CI#19
HanXHX merged 1 commit intomasterfrom
fix_ci_deprecations

Conversation

@HanXHX
Copy link
Copy Markdown
Contributor

@HanXHX HanXHX commented Mar 12, 2026

Copilot AI review requested due to automatic review settings March 12, 2026 14:58
Copy link
Copy Markdown
Contributor

Copilot AI left a 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 updates GitHub Actions workflows to force JavaScript-based actions to run on Node.js 24, aligning CI/release automation with GitHub-hosted runner deprecations of Node 20.

Changes:

  • Add FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true at the workflow level for CI, Docker, release, and Docker Hub description workflows.
  • Ensure the reusable workflows (_ci.yml, _docker.yml) also set the variable so callers are covered.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
.github/workflows/release.yml Forces Node 24 for release workflow jobs (including GoReleaser action usage).
.github/workflows/dockerhub-description.yml Forces Node 24 for the Docker Hub description update action.
.github/workflows/_docker.yml Forces Node 24 inside the reusable Docker publish workflow.
.github/workflows/_ci.yml Forces Node 24 inside the reusable CI workflow (checkout/setup/lint actions).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Removes warning about Node 20 deprecation
@HanXHX HanXHX force-pushed the fix_ci_deprecations branch from 3965f37 to 56f1b46 Compare March 12, 2026 15:15
@HanXHX HanXHX merged commit fcf3aab into master Mar 12, 2026
4 checks passed
@HanXHX HanXHX deleted the fix_ci_deprecations branch March 12, 2026 20:00
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