Skip to content

ci: bump checkout and codeql-action off deprecated Node 20 - #3418

Open
aresnik11 wants to merge 2 commits into
mainfrom
ajr-fix-ci-warning
Open

ci: bump checkout and codeql-action off deprecated Node 20#3418
aresnik11 wants to merge 2 commits into
mainfrom
ajr-fix-ci-warning

Conversation

@aresnik11

@aresnik11 aresnik11 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Overview

Bumps GitHub Actions off the deprecated Node.js 20 runtime to stop the CI warning appearing on gamut PRs:

Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4.

Changes

  • actions/checkout — standardized all 12 usages across the workflows to the SHA-pinned v6.0.2 (de0fac2e…) already used in test.yml and throughout the mono repo. Replaces the older @v4 (8) and @v2 (2) tags.
  • github/codeql-action — bumped init/autobuild/analyze from @v1@v3. v1 is fully retired (GitHub rejects analysis uploads from it), so this also restores a working CodeQL scan. Kept the existing floating-tag style since these weren't SHA-pinned before.

Why

  • The Node 20 fallback GitHub applies today is temporary — once Node 20 is removed from runners, actions/checkout@v4 will fail to start instead of warning, breaking every gamut workflow (including release/publish) at once. See the deprecation notice.
  • Low-risk: runners are ubuntu-24.04 (Node 24 supported), checkout v5/v6 is a drop-in, and the CodeQL language is javascript (interpreted) so autobuild is a no-op.

PR Checklist

  • Related to designs:
  • Related to JIRA ticket: [ABC-123]
  • Version plan added/updated (or not needed)
  • I have run this code to verify it works
  • This PR includes unit tests for the code change
  • This PR includes testing instructions tests for the code change
  • The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

CI is the test surface here — no app/runtime behavior changes.

  1. Warning is gone — on this PR's CI run, confirm no jobs show the "Node.js 20 is deprecated … actions/checkout" annotation.
  2. Test suite passes — the existing Test Suite and push workflows run green (checkout + build + tests still work with the pinned version).
  3. CodeQL runs — the CodeQL workflow's Initialize/Analyze steps complete successfully on @v3 (and no longer emit a Node deprecation warning). Note: CodeQL also runs on a schedule and on main, so a green run here confirms the init/analyze upgrade.

PR Links and Envs

Repository PR Link
Monolith Monolith PR
Mono Mono PR

Standardize actions/checkout to the SHA-pinned v6.0.2 already used in
test.yml (and mono), and bump github/codeql-action to v3, to clear the
Node 20 deprecation warning on CI runs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nx-cloud

nx-cloud Bot commented Aug 19, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 63f8785


☁️ Nx Cloud last updated this comment at 2026-08-20 15:47:08 UTC

@aresnik11
aresnik11 marked this pull request as ready for review August 19, 2026 18:38
@aresnik11
aresnik11 requested a review from a team as a code owner August 19, 2026 18:38
@github-actions

Copy link
Copy Markdown
Contributor

@codecademydev

Copy link
Copy Markdown
Collaborator

📬 Published Alpha Packages:

Package Version npm Diff
@codecademy/gamut 73.0.1-alpha.817a48.0 npm diff
@codecademy/gamut-icons 10.0.1-alpha.817a48.0 npm diff
@codecademy/gamut-illustrations 1.0.1-alpha.817a48.0 npm diff
@codecademy/gamut-kit 3.0.18-alpha.817a48.0 npm diff
@codecademy/gamut-patterns 1.0.1-alpha.817a48.0 npm diff
@codecademy/gamut-styles 21.0.1-alpha.817a48.0 npm diff
@codecademy/gamut-tests 7.0.1-alpha.817a48.0 npm diff
@codecademy/variance 1.0.1-alpha.817a48.0 npm diff
eslint-plugin-gamut 3.0.1-alpha.817a48.0 npm diff

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