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

Add package-lock.json to source control #234

Closed
seebees opened this issue Nov 14, 2019 · 0 comments · Fixed by #235
Closed

Add package-lock.json to source control #234

seebees opened this issue Nov 14, 2019 · 0 comments · Fixed by #235

Comments

@seebees
Copy link
Contributor

seebees commented Nov 14, 2019

As of v3.18.2, as discussed in lerna/lerna#1998,
lerna will now force-add the package-lock.json file in each package
to git as part of the versioning workflow.
If these files are in our gitignore this will result in a confusing developer experience
and a failure of the lerna version command.

Further clearly identifying the version of all packages for a given commit will help with debugging.
package-lock.json was initially ignored because this is a library that is used by other developers
and the lock file is ignored when this package was published
and installed as a dependency.

seebees added a commit to seebees/aws-encryption-sdk-javascript that referenced this issue Nov 14, 2019
resolves aws#234

Lerna version 3.18.2 will version package-lock.json files.
This means that if they are in .gitingnore the version command will fail.
Better visibility into versions should help debugging.
seebees added a commit that referenced this issue Nov 20, 2019
resolves #234

Lerna version 3.18.2 will version package-lock.json files.
This means that if they are in .gitingnore the version command will fail.
Better visibility into versions should help debugging.
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 a pull request may close this issue.

1 participant