Skip to content

ci: Update Node and action versions for PR builds#480

Merged
kadler merged 2 commits into
mainfrom
update-pr-ci
Jul 2, 2026
Merged

ci: Update Node and action versions for PR builds#480
kadler merged 2 commits into
mainfrom
update-pr-ci

Conversation

@kadler

@kadler kadler commented Jul 2, 2026

Copy link
Copy Markdown
Member

Adjust PR workflow to address Node 20 being deprecated within GitHub actions.

  • Node 20 fails on windows-latest and is EOL, update to Node 22.
  • Update actions/checkout and actions/setup-node to latest versions which now use Node 24.

https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners

@kadler kadler requested a review from abmusse July 2, 2026 17:39
@kadler kadler changed the title ci: Update Node version and action versions ci: Update Node and action versions for PR builds Jul 2, 2026
@kadler

kadler commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

Ok, so the problem wasn't Node 20, but that windows-latest now uses VS2026: https://github.blog/changelog/2026-05-14-github-actions-upcoming-image-migrations/#windows-2025-visual-studio-2026-image-migration

This breaks Node 22 as well, so we need to switch to windows-2022 regardless (which we also use for release builds). Perhaps it's best to just fix the issue at hand and have separate PR for dropping Node 20 support which can also update the readme, etc.

kadler added 2 commits July 2, 2026 13:35
windows-latest now uses VS 2026 [1], which Node 20 doesn't understand.

  find VS unknown version "undefined" found at
  "C:\Program Files\Microsoft Visual Studio\18\Enterprise"

Switch to windows-2022 to continue to use VS 2022.

[1] https://github.blog/changelog/2026-05-14-github-actions-upcoming-image-migrations/#windows-2025-visual-studio-2026-image-migration
Node 20 is deprecated within GitHub Actions [1]. Update actions/checkout
and actions/setup-node to latest versions which now use Node 24.

[1] https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners

@abmusse abmusse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@kadler kadler merged commit 23ef543 into main Jul 2, 2026
4 checks passed
@kadler kadler deleted the update-pr-ci branch July 2, 2026 21:29
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