Skip to content

fix(release): publish on Node LTS so npm@latest installs - #1679

Merged
lucasmcdonald3 merged 2 commits into
v4.xfrom
fix-v4.x-publish-node
Jul 22, 2026
Merged

fix(release): publish on Node LTS so npm@latest installs#1679
lucasmcdonald3 merged 2 commits into
v4.xfrom
fix-v4.x-publish-node

Conversation

@lucasmcdonald3

@lucasmcdonald3 lucasmcdonald3 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

Bump the publish job's Node to always publish with lts/*.

The latest npm (12.x) requires Node ^22 so 20 fails, but it probably makes sense to always use LTS to publish.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Check any applicable:

  • Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

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.

Pull request overview

Updates the production release workflow to publish packages using the newest Node runtime so that installing npm@latest during publishing succeeds (since newer npm versions require newer Node majors).

Changes:

  • Switch the publish job’s actions/setup-node version from Node 20 to latest.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/prod-release.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@lucasmcdonald3 lucasmcdonald3 changed the title fix(release): publish on Node latest so npm@latest installs fix(release): publish on Node LTS so npm@latest installs Jul 22, 2026
@lucasmcdonald3
lucasmcdonald3 merged commit 84b753f into v4.x Jul 22, 2026
33 checks passed
@lucasmcdonald3
lucasmcdonald3 deleted the fix-v4.x-publish-node branch July 22, 2026 21:28
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