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

Support yarn #343

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Support yarn #343

wants to merge 1 commit into from

Conversation

vladimyr
Copy link

@vladimyr vladimyr commented Jun 5, 2019

This resolves #195 by supporting following scenario:

$ NPM_CHECK_INSTALLER=yarn npm-check

Also if yarn's lockfile has been found it will be used as default package manager 🎉

@vladimyr
Copy link
Author

vladimyr commented Jun 5, 2019

Those build failures are due to node@4 lacking es6 support...

@hacknug
Copy link

hacknug commented Sep 18, 2019

@dylang any chance this could be reviewed and merged? For some reason, yarn upgrade-interactive isn't as reliable for me as your package 🤷‍♂

@VincentLanglet
Copy link

VincentLanglet commented Nov 2, 2019

@dylang This would be great

Edit: It seems there is another PR https://github.com/dylang/npm-check/pull/365/files

@dkhunt27
Copy link
Contributor

Merged with #397

@fschoenfeldt
Copy link

fschoenfeldt commented Jul 28, 2021

By the way, by using NPM_CHECK_INSTALLER=yarn npm-check, yarn tries to install via yarn install which doesn't work anymore:

? Choose which packages to update. [email protected], [email protected], [email protected], babel-plugin-styled-compo
[email protected], [email protected], [email protected], [email protected], [email protected], [email protected], gatsby-plugi
[email protected], [email protected], [email protected], [email protected], gatsby-plugin-styled-compon
[email protected], [email protected], [email protected], [email protected], [email protected],
 [email protected], [email protected], [email protected]

$ yarn install --save [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] --color=always
Command failed: yarn install --save [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] --color=always
warning ../../package.json: No license field
error `install` has been replaced with `add` to add new dependencies. Run "yarn add [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]" instead.

For more detail, add `--debug` to the command

For now, I just run the suggested yarn add ... command. 😄

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 this pull request may close these issues.

Support yarn as an alternative to npm
5 participants