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

[BUG] run npm update --save results in error if dependency is defined with file: #4454

Closed
2 tasks done
alex-enchi opened this issue Feb 22, 2022 · 0 comments · Fixed by #4617
Closed
2 tasks done

[BUG] run npm update --save results in error if dependency is defined with file: #4454

alex-enchi opened this issue Feb 22, 2022 · 0 comments · Fixed by #4617
Assignees
Labels
Bug thing that needs fixing Priority 1 high priority issue Release 8.x work is associated with a specific npm 8 release

Comments

@alex-enchi
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

This issue exists in the latest npm version

  • I am using the latest npm

Current Behavior

We have one of the local dependencies defined with file:../dependency Run of npm update --save results in

npm ERR! Cannot read properties of null (reading 'isRoot')

and package.json is not updated

Expected Behavior

update should skip file:../ dependencies and update properly package.json

Steps To Reproduce

just having one of dependencies defined as file:../ and run of npm update --save results in error

Environment

  • npm: 8.5.1
  • Node.js: v16.14.0
  • OS Name: MacOs
  • System Model Name:
  • npm config: nothing worth mentioning
@alex-enchi alex-enchi added Bug thing that needs fixing Needs Triage needs review for next steps Release 8.x work is associated with a specific npm 8 release labels Feb 22, 2022
@fritzy fritzy added Priority 1 high priority issue and removed Needs Triage needs review for next steps labels Feb 22, 2022
@nlf nlf self-assigned this Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug thing that needs fixing Priority 1 high priority issue Release 8.x work is associated with a specific npm 8 release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants