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

npx run existing bins #1906

Closed
wants to merge 4 commits into from
Closed

npx run existing bins #1906

wants to merge 4 commits into from

Conversation

nlf
Copy link
Contributor

@nlf nlf commented Oct 6, 2020

This restores the old npx foo behavior of looking for foo in the local and global bin paths first, before assuming that it's a package name and attempting to install it.

Related to: #1845 and #1746

@nlf nlf requested a review from a team as a code owner October 6, 2020 21:11
@isaacs
Copy link
Contributor

isaacs commented Oct 6, 2020

LGTM!

@nlf nlf added the Release 7.x work is associated with a specific npm 7 release label Oct 9, 2020
ruyadorno pushed a commit that referenced this pull request Oct 9, 2020
@nlf nlf closed this Oct 9, 2020
@jsg2021
Copy link

jsg2021 commented Oct 10, 2020

How would we force always installing & running a package? (esp if the existing command isn't what I want to run)

@isaacs
Copy link
Contributor

isaacs commented Oct 13, 2020

@jsg2021 npx -p some-package do-something, same as with npm v6.

@jsg2021
Copy link

jsg2021 commented Oct 13, 2020

@isaacs do i have to add --yes to skip the install prompt for automation?

@jsg2021
Copy link

jsg2021 commented Oct 13, 2020

I currently have an automation making a call npx --ignore-existing @scope/command in npm6. That would always download and install @scope/command@latest and run command. I'm not seeing a direct translation.

@jsg2021
Copy link

jsg2021 commented Oct 13, 2020

I think I'm actually hitting a bug.
#1955

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release 7.x work is associated with a specific npm 7 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants