search

yeoman / doctor

Detect potential issues with users system that could prevent Yeoman from working correctly
http://yeoman.io
BSD 2-Clause "Simplified" License
61 stars 17 forks source link

npm version should be 2.x #31

Closed jmfrancois closed 8 years ago

jmfrancois commented 8 years ago

If you look at this commit https://github.com/yeoman/doctor/commit/c9c6a2370a336ab81777772d017a7bc8ef90a2b7 it means yeoman need npm3. But nodejs LTS is 4.X and this means npm is 2.X not 3.X

Should we revert partially this change and ask for node 4 and npm 2 ?

sindresorhus commented 8 years ago

No, that's intentional. npm@2 has lots of problems. You can easily upgrade to latest npm version with npm i -g npm.

jmfrancois commented 8 years ago

I don't understand the point. Every software has problems even npm3. A requirement mean it can't work with npm2 or I don't think this is the case. AKFAIK yeoman do not use internals of npm that should need npm3.

SBoudrias commented 8 years ago

It's not a requirement, it's a warning.

We receive hundreds of bug ticket all the time because of people hitting npm 2 bugs. This was too much work for us, so we set a warning on older npm versions.