-
Notifications
You must be signed in to change notification settings - Fork 395
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
Migrate from yarn to pnpm #4426
Comments
Would like to work on this. Is this issue ready to work on? |
@harshsbhat Sure, this is available. Thanks in advance! |
Hi @nolanlawson , I hope you're doing well! I’ve attempted to migrate to pnpm, but after running pnpm install for some dependencies, I encountered the following error:
It seems that @lwc/lwc-modules-bar is not available in the npm registry, or I may not have the necessary permissions to fetch it. Additionally, no authorization header was set for the request. Thank you for your help! |
That's not a real module, it's just used in a test fixture. You'll need to configure pnpm to ignore package.json files found in any lwc/packages/@lwc/module-resolver/src/__tests__/fixtures/mapping/package.json Lines 7 to 8 in 3ca1070
Also, seems weird to be using pnpm, but using the yarn registry. 🤷 |
We are still using yarn v1 in this repo – we should migrate to a more modern solution.
After internal discussion we decided on pnpm. Two challenges:
yarn.lock
to test for thisThe text was updated successfully, but these errors were encountered: