GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix(libnpmexec): fix bug not install latest pkg by jihunleekr · Pull Request #4929 · npm/cli

Conversation

@jihunleekr

@jihunleekr jihunleekr commented May 22, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

After the major version of the package is changed (e.g. 0.9 > 1.0 > 2.0)
npx just asks if you want to install it every time
It doesn't actually install.

References

Fixes #4871
Fixes #4619

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

slorber reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@jihunleekr
fix(libnpmexec): fix bug not install latest pkg
e255593
@jihunleekr
jihunleekr requested a review from a team as a code owner May 22, 2022 20:59
ljharb
ljharb reviewed May 22, 2022
View reviewed changes
Comment thread workspaces/libnpmexec/lib/index.js Resolved

Uh oh!

There was an error while loading. Please reload this page.

@jihunleekr @ljharb
chore: simplify code
f68ef8e
Co-authored-by: Jordan Harband <[email protected]>
@jihunleekr
jihunleekr requested a review from ljharb May 23, 2022 05:31
@ljharb
ljharb removed their request for review May 23, 2022 15:13
@jihunleekr jihunleekr mentioned this pull request May 26, 2022
npx 사용시 매번 psoj 를 설치할건지 물어본다면.. jihunleekr/psoj#2
Open
ruyadorno
ruyadorno approved these changes Jun 1, 2022
View reviewed changes

@ruyadorno ruyadorno left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's go ahead with this change, I'm pretty sure this will be a more reasonable behavior in terms of UX 👍 thanks @jihunleekr

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

jihunleekr reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@ruyadorno
ruyadorno merged commit fcc72dd into npm:latest Jun 1, 2022
@github-actions github-actions Bot mentioned this pull request Jun 1, 2022
chore(latest): release libnpmexec 4.0.6 #4976
Merged
@ruyadorno ruyadorno mentioned this pull request Jun 1, 2022
Release/v8.12.0 #4977
Merged
@npm-cli-bot npm-cli-bot mentioned this pull request Jun 1, 2022
deps: upgrade npm to 8.12.0 nodejs/node#43291
Closed
@ruyadorno ruyadorno mentioned this pull request Jun 2, 2022
[BUG] npm init <package> doesn't use newer version if existing version is semver-major incompatible #3444
Closed
1 task done
@jihunleekr
jihunleekr deleted the fix-chk-ver branch June 2, 2022 05:17
@ruyadorno ruyadorno mentioned this pull request Jun 2, 2022
deps: upgrade npm to 8.12.1 nodejs/node#43301
Merged
@wraithgar wraithgar mentioned this pull request Jun 28, 2022
[BUG] npx package@version uses incorrect installed version in npx 7 #3210
Closed
@github-actions github-actions Bot mentioned this pull request Aug 10, 2022
chore(latest): release libnpmexec 4.0.10 #5275
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@ljharb ljharb ljharb left review comments
@ruyadorno ruyadorno ruyadorno approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] npx command not install latest package [BUG] npx add confirmation to proceed everytime

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@jihunleekr @ljharb @ruyadorno
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.