Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update pnpm version and pin in package.json #946

Merged
merged 5 commits into from
Aug 23, 2024

Conversation

Parker-Stafford
Copy link
Contributor

https://github.com/pnpm/pnpm/releases/tag/v9.7.0
pnpm/pnpm#8363

In order for the packageManger field to be picked up global pnpm must be >=9.7.0.

After that when you're in the js directory any pnpm commands will use the version specified in package json

@Parker-Stafford Parker-Stafford requested a review from a team as a code owner August 22, 2024 21:29
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 22, 2024
@@ -26,7 +26,7 @@ jobs:
- name: Setup PNPM
uses: pnpm/action-setup@v4
with:
version: 8
version: 9.8.0
Copy link
Contributor

Choose a reason for hiding this comment

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

this has to be specified?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it doesnt have to specify the fullpath but figured we might as well keep it in sync with whats in pacakge.json, which does have to specify the whole path

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 22, 2024
@Parker-Stafford Parker-Stafford merged commit 5a63aa0 into main Aug 23, 2024
3 checks passed
@Parker-Stafford Parker-Stafford deleted the parker/update-pnpm-version branch August 23, 2024 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants