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

WARNING: npm install --global yarn #464

Open
GianClaudioScarafini opened this issue Apr 14, 2024 · 4 comments · May be fixed by #469
Open

WARNING: npm install --global yarn #464

GianClaudioScarafini opened this issue Apr 14, 2024 · 4 comments · May be fixed by #469

Comments

@GianClaudioScarafini
Copy link

ISSUE
within the readme file, you have instructed to use :

npm install --global yarn

this returns a WARN :
npm WARN config global --global, --local are deprecated.

@GianClaudioScarafini
Copy link
Author

SOLUTION
this might be the solution:
npm install yarn --location=global

@ajdubovoy
Copy link
Contributor

Hm (looping in @Eschults here). I checked the yarn docs and they actually fully recommend against installing globally now.

https://yarnpkg.com/corepack

Instead, it now comes as part of Corepack. So, they say to run:

corepack enable

With all node versions >14, this should come by default. But if you get any errors, you can run npm install -g corepack to install it.

So, do we feel OK if I update setup to recommend this? I'm a bit scared to do so, since it's hard for me to test, since I don't have a blank system to try running it on from scratch. I guess if we want I could spin out a Digital Ocean droplet and try running our Ubuntu setup on it with this command and see if it works?

@Eschults
Copy link
Member

👋 @ajdubovoy, OK to update the setup w/ the latest Yarn recommendations, for this kind of short tests on different systems I would suggest involving senior teachers of our community and kindly ask them if they can run the commands, Engineering has very low bandwidth and cannot provide you w/ this kind of support at the minute 🙏

@ajdubovoy
Copy link
Contributor

Ok sounds good!

ajdubovoy added a commit that referenced this issue May 22, 2024
@ajdubovoy ajdubovoy linked a pull request May 22, 2024 that will close this issue
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 a pull request may close this issue.

3 participants