Skip to content

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

Merged
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-s merged 4 commits into
microsoft:mainfrom
yury-s:publish-node-lts

Conversation

@yury-s

@yury-s 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.

yury-s added 2 commits July 9, 2026 12:05
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-s yury-s changed the title devops: use node lts for publishing devops: use node lts/* for non-test workflows Jul 9, 2026
yury-s added 2 commits July 9, 2026 12:29
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

github-actions Bot commented Jul 9, 2026

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

github-actions Bot commented Jul 9, 2026

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-s merged commit 32e8fd9 into microsoft:main Jul 9, 2026
46 of 48 checks passed
@yury-s yury-s deleted the publish-node-lts branch July 9, 2026 20:34
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