Skip to content

devops: use node lts/* for non-test workflows - #41716

Merged
Yury Semikhatsky (yury-s) merged 4 commits into
microsoft:mainfrom
yury-s:publish-node-lts
Jul 9, 2026
Merged

devops: use node lts/* for non-test workflows#41716
Yury Semikhatsky (yury-s) merged 4 commits into
microsoft:mainfrom
yury-s:publish-node-lts

Conversation

@yury-s

@yury-sYury Semikhatsky (yury-s) commented Jul 9, 2026

Copy link
Copy Markdown
Member

Summary

  • Switch non-test workflows to node-version: lts/* so they track the current LTS instead of pinning specific node majors: publish (release, extension, docker), roll (browser, nodejs, stable test runner), fix-flakes, triage, report, infra and copilot setup.
  • Test workflows (tests_*.yml) are left untouched since they deliberately pin node versions to exercise compatibility across majors.
  • The current LTS (node 24) bundles npm >= 11.5.1, so OIDC publishing keeps working with no separate npm upgrade step.

Track the current LTS via `lts/*` rather than pinning a specific major.
The current LTS (node 24) bundles npm >= 11.5.1, so OIDC publishing keeps
working without a separate npm upgrade step.
Track the current LTS in publish, roll, triage, report and infra
workflows instead of pinning specific node majors. Test workflows keep
their explicit versions since they exercise node compatibility.
@yury-sYury Semikhatsky (yury-s) changed the title devops: use node lts for publishingdevops: use node lts/* for non-test workflowsJul 9, 2026
This job commits package-lock.json. Writing it with a newer npm (from
lts/*) can churn the lockfile against contributors running an older npm
locally, so pin the node major instead.
The repo pins no local node/npm baseline (no .nvmrc/packageManager), so
there is no canonical npm to match; keep this workflow consistent with
the other non-test workflows on lts/*.
@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

2 failed
❌ [msedge] › mcp/annotate.spec.ts:110 › should abort annotation when last screenshot is removed @mcp-windows-latest-msedge
❌ [webkit] › mcp/annotate.spec.ts:497 › should disengage annotate mode when --annotate client disconnects @mcp-macos-latest-webkit

7733 passed, 1248 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

2 flaky⚠️ [chromium-library] › library/video.spec.ts:717 › screencast › should work with video+trace `@chromium-ubuntu-22.04-arm-node20`
⚠️ [firefox-library] › library/heap.spec.ts:223 › should not leak workers `@firefox-ubuntu-22.04-node20`

49514 passed, 1161 skipped


Merge workflow run.

@yury-s
Yury Semikhatsky (yury-s) merged commit 32e8fd9 into microsoft:mainJul 9, 2026
46 of 48 checks passed
@yury-s
Yury Semikhatsky (yury-s) deleted the publish-node-lts branch July 9, 2026 20:34
Sign up for freeto 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

@yury-s@pavelfeldman@dgozman