ci(release): drop pnpm with.version so packageManager wins#176
Conversation
There was a problem hiding this comment.
Pull request overview
Removes the explicit version: 9 input from the pnpm/action-setup@v4 step in the publish workflow so the packageManager field in package.json (pnpm@9.15.9) is the sole source of truth, resolving the ERR_PNPM_BAD_PM_VERSION failure on main.
Changes:
- Drop
with.version: 9from the Setup PNPM step in.github/workflows/release.yml.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Heads up — the Publish workflow downstream of this PR (run https://github.com/GSTJ/react-native-magic-modal/actions/runs/26082273487) failed at the npm publish step with HTTP 403. Root cause: the current Fix: rotate Also opened #179 to pre-stamp the version at |
Summary
Hotfix for the 🚀 Publish workflow on
main, which has failed twice consecutively after PR #166 was merged (at 05:39 UTC).The error is not an npm auth issue but a pnpm version conflict:
The prior orchestrator commit (
479ad40) bumpedpnpm/action-setupfromv2/version: 8tov4/version: 9but leftwith.versionin place, conflicting withpackageManager: pnpm@9.15.9inpackage.json.This PR drops
with.versionentirely sopackageManagerwins — mirroring the successful fix applied tobranch-validation.ymlin commitfa0675c.Test plan
mainafter mergenpm view react-native-magic-modal versionshows the new 7.x release