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: pnpm version control by corepack #303

Closed
wants to merge 3 commits into from
Closed

Conversation

u-yas
Copy link

@u-yas u-yas commented Aug 24, 2023

Summary/ Motivation (TLDR;)

When I set up jest-preview on my local environment, it changed the lockfile version in pnpm-lock.yml. When I looked it up, jest-preview's pnpm was v7. The current latest version of pnpm is v8.6.12.
I made it possible to version control pnpm by utilizing the function by corepack. This is also how repositories like next.js and react-hook-form manage it.

https://github.com/vercel/next.js/blob/canary/contributing/core/developing.md#developing

https://github.com/react-hook-form/react-hook-form/blob/master/package.json#L147

Chores

  • pnpm version control by corepack.
  • Updated pnpm-lock.yml
  • Updated setup-pnpm of github actions

*This is because the version is automatically determined by referring to the packageManager property of package.json automatically.
https://github.com/pnpm/action-setup/tree/v2.4.0/#version
@netlify
Copy link

netlify bot commented Aug 24, 2023

Deploy Preview for jest-preview-chinese canceled.

Name Link
🔨 Latest commit 34425a8
🔍 Latest deploy log https://app.netlify.com/sites/jest-preview-chinese/deploys/64e77a1d47373100087f0695

@netlify
Copy link

netlify bot commented Aug 24, 2023

Deploy Preview for jest-preview-library canceled.

Name Link
🔨 Latest commit 34425a8
🔍 Latest deploy log https://app.netlify.com/sites/jest-preview-library/deploys/64e77a1d02804c00073ef483

@u-yas u-yas changed the title Bump pnpm chore: pnpm version control by corepack Aug 24, 2023
@u-yas u-yas closed this Aug 31, 2024
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.

1 participant