Skip to content

chore: use staged publishing - #21

Merged
phryneas merged 1 commit into
mainfrom
pr/staged-publish
Oct 27, 2026
Merged

phryneas merged 1 commit into
mainfrom
pr/staged-publish

Conversation

@phryneas

@phryneas phryneas commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

It's okay if a release/tag are created and staged publishing is then rejected. I can always clean up by hand. Staged publishing is worth the hassle.

@pkg-pr-new

pkg-pr-new Bot commented Oct 27, 2026

Copy link
Copy Markdown
undefined https://pkg.pr.new/testing-library/react-render-stream-testing-library/@testing-library/react-render-stream@21

commit: 3e9388d

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.

🟢 Approval recommended

The focused wrapper and workflow changes preserve command arguments and have no identified blocking issues.

No open issues 🎉

What changed in this PR

This PR routes automated releases through npm staged publishing while allowing release and tag creation before staging approval.

Changes:

  • Add an npm wrapper that redirects publish to npm stage publish and forwards other commands unchanged.
  • Update the release job to Node.js >=24.18.0 <25 and add the wrapper to PATH.
File Description
scripts/​wrap-staged/​npm Redirects publishing to staging and removes its own PATH entry to avoid recursion.
.github/​workflows/​ci.yml Updates the release job’s Node.js version and enables the npm wrapper.

Review effort: Balanced

@phryneas
phryneas merged commit b4c7a9c into main Oct 27, 2026
19 checks passed
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