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

Apply npm install to package-lock.json #6517

Closed
wants to merge 1 commit into from

Conversation

noahsilas
Copy link
Contributor

It's not clear what happened in #6478, but it appears that it dropped
the ws dependency of the parse package when it was updating the
local version of the dependency. This means that running npm install
dirties the working tree, which is surprising!

A similar problem seems to have occurred in #6504, that time related to
the commander package.

I've filed an issue for the surprising behavior from greenkeeper:
greenkeeperio/greenkeeper-lockfile#250

It's not clear what happened in parse-community#6478, but it appears that it dropped
the `ws` dependency of the `parse` package when it was updating the
local version of the dependency. This means that running `npm install`
dirties the working tree, which is surprising!

A similar problem seems to have occurred in parse-community#6504, that time related to
the `commander` package.

I've filed an issue for the surprising behavior from greenkeeper:
greenkeeperio/greenkeeper-lockfile#250
@acinader
Copy link
Contributor

probably an npm version mismatch

@dplewis
Copy link
Member

dplewis commented Apr 3, 2020

Closing as will be addressed in #6560

@dplewis dplewis closed this Apr 3, 2020
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.

3 participants