ci: restore pnpm 9 install for publishing - #433
Conversation
|
There was a problem hiding this comment.
🟡 Changes recommended
The publish workflow still appears misaligned with documented/project CI conventions (Node version and tag trigger pattern), which can prevent publishing or produce inconsistent release artifacts.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR updates the npm publish GitHub Actions workflow to restore the publishing install setup by switching pnpm back to v9 and relaxing the install command.
Changes:
- Set
pnpm/action-setupversion back to pnpm9in the publish workflow. - Change dependency installation from
pnpm install --frozen-lockfiletopnpm install.
File summaries
| File | Description |
|---|---|
| .github/workflows/publish.yml | Restores pnpm v9 usage and adjusts install behavior in the publish pipeline. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| uses: pnpm/action-setup@v4 | ||
| with: | ||
| version: 10 | ||
| version: 9 |
| - name: Install pnpm | ||
| uses: pnpm/action-setup@v4 | ||
| with: | ||
| version: 10 | ||
| version: 9 |
PR includes
Screenshot