🔧 Move to devEngines.packageManager#6996
Conversation
Contrary to my first attempt of move done by Claude at #6957, this one has been achieved fully manually and from a Windows machine. We expect it will pass.
This reverts commit d0106a6.
Co-authored-by: Nicolas DUBIEN <[email protected]>
## Description This PR updates the CI/CD workflows to use `pnpm` instead of `npm` for publishing packages to NPM registry. The changes include: 1. **Preview generation**: Added `--pnpm` flag to `pkg-pr-new publish` commands to ensure pnpm is used for preview builds 2. **Package publishing**: Replaced `npm publish` with `pnpm publish --no-git-checks` across all 7 package publishing jobs (fast-check, ava, jest, packaged, poisoning, vitest, worker) 3. **pnpm setup**: Added explicit `pnpm/action-setup` step to all publishing jobs to ensure pnpm is available before Node.js setup 4. **Line number updates**: Updated zizmor.yml ignore rules to reflect the new line numbers after adding pnpm installation steps These changes ensure consistency with the project's use of pnpm as the package manager and prevent potential issues with npm's git checks during CI publishing. ## Checklist - [ ] I have a full understanding of every line in this PR — whether the code was hand-written, AI-generated, copied from external sources or produced by any other tool - [ ] I flagged the impact of my change (minor / patch / major) either by running `pnpm run bump` or by following the instructions from the changeset bot - [ ] I kept this PR focused on a single concern and did not bundle unrelated changes - [ ] I followed the [gitmoji](https://gitmoji.dev/) specification for the name of the PR, including the package scope (e.g. `🐛(vitest) Something...`) when the change targets a package other than `fast-check` - [ ] I added relevant tests and they would have failed without my PR (when applicable) https://claude.ai/code/session_01DdKMHpjQaQjZ3i6o5qd95q --------- Co-authored-by: Claude <[email protected]>
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
@fast-check/ava
fast-check
@fast-check/jest
@fast-check/packaged
@fast-check/poisoning
@fast-check/vitest
@fast-check/worker
commit: |
…conflicts-TNHJ6 # Conflicts: # .github/workflows/build-status.yml # .github/zizmor.yml
cd5aa82 to
0dd3369
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6996 +/- ##
=======================================
Coverage 94.78% 94.78%
=======================================
Files 212 212
Lines 5832 5832
Branches 1535 1535
=======================================
Hits 5528 5528
Misses 296 296
Partials 8 8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Contrary to my first attempt of move done by Claude at #6957, this one has been achieved fully manually and from a Windows machine.
We expect it will pass.