Skip to content

ci: move the AI-disclosure check off the deprecated Node 20 runtime - #227

Merged
co-l merged 1 commit into
co-l:developfrom
quentin452:ci/github-script-v8
Aug 12, 2026
Merged

ci: move the AI-disclosure check off the deprecated Node 20 runtime#227
co-l merged 1 commit into
co-l:developfrom
quentin452:ci/github-script-v8

Conversation

@quentin452

Copy link
Copy Markdown
Contributor

Summary

Every PR run currently prints:

Node 20 is being deprecated. This workflow is running with Node 24 by default. If you need to
temporarily use Node 20, you can set the ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true
environment variable.

It comes from actions/github-script@v7, which is a Node 20 action. v8 is the same action on
Node 24.

No behaviour change. The script body is untouched; context.payload.pull_request and
core.setFailed are unchanged across the major.

Not in this PR, on purpose

actions/checkout@v4 is a Node 20 action for the same reason and is left alone. checkout@v5
changes more than the runtime, so it deserves a review of its own rather than riding along with a
one-line bump.

Verification

The change is one line in .github/workflows/pr.yml. The ai-disclosure job on this PR is the test:
if it runs and evaluates this PR's body, the action works on the new major.

AI-Enhanced Development

Tell what models helped shape this PR:

  • AI Models: Claude Opus 5 (1M context), through Claude Code

Cache Impact

Does this PR affect anything cached — system prompts, tool definitions, skills, or other context?

  • No

🤖 Generated with Claude Code

https://claude.ai/code/session_01EhMynpFbWTNFcyUnobBGZE

actions/github-script@v7 runs on Node 20, which GitHub has deprecated: every
run of this job now prints the deprecation notice, and the runtime will stop
being available on a date GitHub chooses rather than one this repository picks.
v8 is the same action on Node 24.

No behaviour change — the script body is untouched, and `context.payload` and
`core.setFailed` are unchanged across the major.

actions/checkout@v4 is on Node 20 for the same reason and is NOT touched here:
checkout@v5 changes more than the runtime, so it deserves its own review rather
than riding along with a one-line bump.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EhMynpFbWTNFcyUnobBGZE
Copilot AI lite review requested due to automatic review settings August 8, 2026 14:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the PR workflow to avoid the deprecated Node 20 runtime warning by moving the AI-disclosure check step to the Node 24-based major of actions/github-script, without changing the script logic.

Changes:

  • Bump actions/github-script from @v7 to @v8 in the ai-disclosure job.
  • Keep the disclosure-check script body unchanged to preserve behavior.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@co-l
co-l merged commit 92a880f into co-l:develop Aug 12, 2026
6 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.

3 participants