Skip to content

ci(release): rely on packageManager for pnpm version#173

Closed
GSTJ wants to merge 2 commits into
mainfrom
fix/release-pnpm-conflict
Closed

ci(release): rely on packageManager for pnpm version#173
GSTJ wants to merge 2 commits into
mainfrom
fix/release-pnpm-conflict

Conversation

@GSTJ
Copy link
Copy Markdown
Owner

@GSTJ GSTJ commented May 19, 2026

Summary

Test plan

  • Merge and confirm the 🚀 Publish workflow runs without the "Multiple versions of pnpm specified" error
  • Confirm 7.0.0 lands on npm

action-setup@v4 errors when both `version:` and root packageManager are
set:
  Error: Multiple versions of pnpm specified
This caused the post-merge publish (run 26078575325) to fail. The
branch-validation workflow already omits `version` and pulls pnpm from
packageManager (pnpm@9.15.9), so doing the same here keeps both
workflows aligned and unblocks 7.0.0 publish.
Copilot AI review requested due to automatic review settings May 19, 2026 05:41
@GSTJ GSTJ enabled auto-merge (squash) May 19, 2026 05:42
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the 🚀 Publish workflow failure caused by pnpm/action-setup@v4 detecting conflicting pnpm versions when both the workflow input version: and the repo root packageManager field are set. It aligns the release workflow with the existing branch-validation workflow by letting pnpm be selected from package.json (pnpm@9.15.9), unblocking the 7.0.0 publish.

Changes:

  • Remove the explicit version: 9 input from pnpm/action-setup@v4 in the release workflow.
  • Rely on the repository’s root packageManager field (pnpm@9.15.9) to determine the pnpm version.

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

@GSTJ
Copy link
Copy Markdown
Owner Author

GSTJ commented May 19, 2026

Superseded by #176 which is authored by GSTJ (trusted-author bypass) and has the same fix.

@GSTJ GSTJ closed this May 19, 2026
auto-merge was automatically disabled May 19, 2026 06:18

Pull request was closed

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