-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Set packageManager #79
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I'm having second thought about this one as yarn will complain if Should we keep that as a trick for fixing our CI instead of pushing it to everybody? |
The complaint is just a warning, right? Or does it error out? |
I have found corepack to be... not a great user experience, under any
condition.
…On Wed, Jun 19, 2024, 04:54 Michał Krassowski ***@***.***> wrote:
The complaint is just a warning, right? Or does it error out?
—
Reply to this email directly, view it on GitHub
<#79 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAALCRES6SHRILQ5OI7NEZDZIFINPAVCNFSM6AAAAABJPRFQGSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZYGI3DEOJXGQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
It unfortunately errors out 😔 as can be seen there |
I agree and I'm really sad that yarn went all-in that experimental feat' that is far from stable between nodeJS versions. |
So I'm gonna revert this one |
This reverts commit c68f45a.
This is set to avoid issue with tools like dependabot that may run a different version of yarn resulting in a different yarn.lock version.