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

Avoid generating package locks instead of ignoring them #1108

Merged
merged 2 commits into from Dec 24, 2017

Conversation

nickmccurdy
Copy link
Member

Fixes npm/npm#17168. Package locks should not be ignored, they should be disabled, otherwise npm version and other commands may fail.

@nickmccurdy
Copy link
Member Author

See #1109

@@ -0,0 +1 @@
package-lock=false
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: Add a newline

Copy link
Member Author

Choose a reason for hiding this comment

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

Whoops, I meant to do this before. Done.

@fl0w
Copy link
Contributor

fl0w commented Dec 19, 2017

LGTM

@codecov
Copy link

codecov bot commented Dec 19, 2017

Codecov Report

Merging #1108 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1108   +/-   ##
=======================================
  Coverage   99.73%   99.73%           
=======================================
  Files           5        5           
  Lines         371      371           
=======================================
  Hits          370      370           
  Misses          1        1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 841844e...25afd6f. Read the comment docs.

@dead-horse dead-horse merged commit 79c3c73 into koajs:master Dec 24, 2017
@nickmccurdy nickmccurdy deleted the package-lock-fix branch December 24, 2017 07:19
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.

None yet

3 participants