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

use npx when package manager is yarn #635

Closed
wants to merge 1 commit into from
Closed

use npx when package manager is yarn #635

wants to merge 1 commit into from

Conversation

jetersen
Copy link
Contributor

@jetersen jetersen commented Jan 10, 2020

alternative to #634

fixes #634
fixes #627

@typicode
Copy link
Owner

Yarn 2 has a new Plug'n'Play features which installs modules in a single pnp.js file and not in node_modules anymore. From what I understand, to be able to run husky-run bin, the package manager needs to be Yarn.

So I don't think that in this context, npx --no-install husky-run ... will work.

https://next.yarnpkg.com/features/pnp

@typicode typicode closed this Jan 10, 2020
@arcanis
Copy link
Contributor

arcanis commented Jan 10, 2020

Yep, exactly 👍

@jetersen
Copy link
Contributor Author

That's fine cannot be aware of all the context :)

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.

[4.0.0 | Git Bash | Windows 10] stdin is not a tty
3 participants