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

update requirements to Node.js 4.2.0 and npm 3.3.0 #26

Closed sindresorhus closed 8 years ago

sindresorhus commented 8 years ago

Node.js 4.2.0 is now LTS, so it's a good target to require. npm 3.3.0 is a stable version of npm.

passy commented 8 years ago

:+1:

kevva commented 8 years ago

Thumbs up from me too.

arthurvr commented 8 years ago

:+1:

eddiemonge commented 8 years ago

this is a big change. Means if people want to use Yeoman, they have to upgrade node/npm and hope everything else they use it with doesn't break. I'm all for that but I'm saying we need to be prepared for a bunch of tickets in response to it.

SBoudrias commented 8 years ago

I still see a lot of issues with Node 4.x and native modules. I know some of our generators uses some that haven't been updated (or we didn't bump for 4.0).

Not sure we should recommend 4.2 before we're sure we're in a good state

sindresorhus commented 8 years ago

I know some of our generators uses some that haven't been updated

Which native dependencies? I'll fix.

SBoudrias commented 8 years ago

From what I could see, Usually related to phantomjs

sindresorhus commented 8 years ago

PhantomJS is not a native dependency in that sense, though. It's just a binary. It doesn't break with new Node.js versions. Do you have any links I could look at?