search

lewagon / setup

Setup instructions for Le Wagon's students on their first day of Web Development Bootcamp
https://www.lewagon.com
18.29k stars 1.6k forks source link

WARNING: npm install --global yarn #464

Open GianClaudioScarafini opened 5 months ago

GianClaudioScarafini commented 5 months ago

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

npm install --global yarn

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

GianClaudioScarafini commented 5 months ago

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

ajdubovoy commented 5 months ago

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 commented 5 months ago

👋 @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 commented 5 months ago

Ok sounds good!