Skip to content

fix: opt into Node.js 24 for ilammy/msvc-dev-cmd - #364

Merged
BenJule merged 1 commit into
masterfrom
fix/nodejs24-msvc-dev-cmd
Jun 1, 2026
Merged

fix: opt into Node.js 24 for ilammy/msvc-dev-cmd#364
BenJule merged 1 commit into
masterfrom
fix/nodejs24-msvc-dev-cmd

Conversation

@BenJule

@BenJule BenJule commented Jun 1, 2026

Copy link
Copy Markdown
Owner

Closes #363

Summary

  • ilammy/msvc-dev-cmd@v1.13.0 uses node20, which is deprecated in GitHub Actions (forced to Node.js 24 on June 16, 2026; removed September 16, 2026)
  • Upstream has no Node.js 24 release yet (Update to Node.js 24 ilammy/msvc-dev-cmd#106 pending)
  • The action's JavaScript is already Node.js 24 compatible — only the action.yml version declaration is outdated
  • Fix: add FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: 'true' to the step env, as recommended by GitHub

Change

.github/workflows/_build-core.yml — adds one env line to the Setup MSVC developer environment step.

Test plan

  • Windows build (windows-2022) succeeds without Node.js deprecation warning
  • Windows ARM build (windows-11-arm) is unaffected (fails on OpenCV, not on this step)

@BenJule BenJule self-assigned this Jun 1, 2026
@BenJule BenJule added this to the v02.07.00-dev milestone Jun 1, 2026
@github-actions github-actions Bot added area: ci GitHub Actions, CI/CD area: ui User interface / wxWidgets platform: windows Windows-specific issue tests Test files and test infrastructure labels Jun 1, 2026
@BenJule
BenJule requested a review from Copilot June 1, 2026 18:26

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@BenJule
BenJule merged commit 1ff77d5 into master Jun 1, 2026
7 of 9 checks passed
@BenJule
BenJule deleted the fix/nodejs24-msvc-dev-cmd branch June 1, 2026 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci GitHub Actions, CI/CD area: ui User interface / wxWidgets platform: windows Windows-specific issue tests Test files and test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: update ilammy/msvc-dev-cmd to Node.js 24 (deadline June 16, 2026)

2 participants