-
Notifications
You must be signed in to change notification settings - Fork 3
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
Plugin stuck on "Ready-to-install package ... Just run your package manager install command." #20
Comments
Thanks for reporting. I have three questions:
Thanks! |
Oddly, it seems to work, a bit. See screenshot below. Moment is flagged OK, but util-dynamodb is still saying to un the install command, even though I have. |
Would it be feasible for you to delete the "node_modules" folder and run "npm i" again? I suspect it's some defect within node_modules that may be causing problems (this usually happens after many updates to npm itself, or when using other package managers). |
I too am experiencing this issue which seemed to appear pretty recently as one of the projects wasn't showing the ''now run your package manager." message but now it is that project and another one I just created. There aren't any error indicators in the vscode output window so it's not clear what's going on. I have updated vscode a few times so perhaps it's related? |
@backtoyoujames I suspect this problem is related to the missing file Another related issue is if you are using However, PR #22 should resolve the first case and add support for |
I'm still having this issue despite having a package-lock.json file and using npm. |
@paul-uz Restarting vscode the problem persists? You can create a reduced case for I test? |
Yup restarted VSCode several times, same thing every time. |
@paul-uz It's a little weird. Can you create a reduced project? Maybe only package.json could help to understand. But if you could generate a reduced project it would help more. So I can see if PR is the solution or even improve it. |
I have this happen on npm workspaces! |
The main package.json doesnt have this error but any inner package.json keeps saying this :p |
You have package.json in a non-root path? Maybe this need to be fixed, because I never do in that way. I will check in this week. |
It's quite a common thing with a monorepo |
maybe we can turn it off? |
Must be fixed now via #22. |
but it still isnt merged 😔 |
Yeah, I just asked @mskelton if he will approves the PR. |
I have published a fork extension called npm-outdated-plus with this changes. |
The plugin correctly detects out dated versions, and updates the number when I click to do so. It then prompts with a message like:
So, I run
npm i
and the update is installed. However, the plugin continues to persist the "Ready-to-install" message.I'm using v2.2.0 and VS Code 1.76.2 on MacOS 11.6.5
The text was updated successfully, but these errors were encountered: