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 gitignore #1455

Merged
merged 1 commit into from Aug 1, 2018

Conversation

@KayleePop
Copy link
Contributor

KayleePop commented Aug 1, 2018

Seems like a good idea since npm automatically creates it.

At the very least it would remove a minor annoyance

Copy link
Member

DiegoRBaquero left a comment

Had this discussion with @feross we ended with globally git ignore package-lock.json, makes sense to put it in there for others.

@DiegoRBaquero DiegoRBaquero merged commit 4e748f2 into webtorrent:master Aug 1, 2018
3 checks passed
3 checks passed
WIP ready for review
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@KayleePop KayleePop deleted the KayleePop:lockfile-ignore branch Aug 1, 2018
@feross

This comment has been minimized.

Copy link
Member

feross commented Aug 4, 2018

Yep, this makes sense.

@lock lock bot locked as resolved and limited conversation to collaborators Nov 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.