AndersDJohnson / use-yarn

Force users to use yarn instead of npm.
https://andersdjohnson.github.io/use-yarn/
ISC License
50 stars 12 forks source link

is this package not worked on yarn v1.0.x versions #2

Closed chilijung closed 7 years ago

chilijung commented 7 years ago

Before v1.0.x version this package worked normal. After upgraded to v1.0.x use-yarn can't detect I'm using yarn.

➜  projects git:(master) yarn
yarn install v1.0.2
$ use-yarn || yarn global add use-yarn
This project uses yarn instead of npm. Please upgrade!
See https://yarnpkg.com.

To install dependencies:
$ yarn install

To install a new package (also adds to package.json):
$ yarn add [packagename]

To upgrade a package:
$ yarn upgrade [packagename]
chilijung commented 7 years ago

After some research, https://github.com/yarnpkg/yarn/pull/4330 after yarn v1.0.1 version npm_lifecycle_script is added in process.env, so I think this package is broken?

chilijung commented 7 years ago

https://github.com/AndersDJohnson/is-npm-not-yarn/pull/1 , this will fix this issue.

AndersDJohnson commented 7 years ago

@chilijung Thanks. Updated and published as 2.0.0.