Skip to content

fix release workflow by removing redundant npm upgrade#33

Merged
teallarson merged 1 commit intomainfrom
teallarson/investigate-failure
Apr 9, 2026
Merged

fix release workflow by removing redundant npm upgrade#33
teallarson merged 1 commit intomainfrom
teallarson/investigate-failure

Conversation

@teallarson
Copy link
Copy Markdown
Collaborator

@teallarson teallarson commented Apr 9, 2026

Summary

  • Removes npm install -g npm@latest from the release workflow — the npm bundled with Node 22.22.2 on GitHub runners is broken (missing promise-retry module), causing the step to fail
  • The default npm from actions/setup-node@v4 is sufficient for npm publish

🤖 Generated with Claude Code

@teallarson teallarson marked this pull request as ready for review April 9, 2026 14:19
@teallarson teallarson closed this Apr 9, 2026
@teallarson teallarson reopened this Apr 9, 2026
@teallarson teallarson changed the title Remove unnecessary npm self-upgrade from release workflow fix release workflow by removing redundant npm installation Apr 9, 2026
@teallarson teallarson changed the title fix release workflow by removing redundant npm installation fix release workflow by removing redundant npm upgrade Apr 9, 2026
@teallarson teallarson force-pushed the teallarson/investigate-failure branch from 24fbd39 to 7722d04 Compare April 9, 2026 14:24
The bundled npm on Node 22.22.2 runners is broken (missing promise-retry),
causing `npm install -g npm@latest` to fail. The default npm from
actions/setup-node is sufficient for publish.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@teallarson teallarson force-pushed the teallarson/investigate-failure branch from 7722d04 to 8c86cd0 Compare April 9, 2026 14:26
Copy link
Copy Markdown
Contributor

@sdserranog sdserranog left a comment

Choose a reason for hiding this comment

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

:shipit:

@teallarson teallarson merged commit 7f7b7f7 into main Apr 9, 2026
@teallarson teallarson deleted the teallarson/investigate-failure branch April 9, 2026 14:53
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