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

Adds package-lock.json to .gitignore #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benjaminhoffman
Copy link

@benjaminhoffman benjaminhoffman commented Dec 22, 2017

Going through the service worker course on udacity and running npm install generates a package.json file thereby automatically creating a diff. May confuse newb users when they run npm reset --hard as there is still a diff.

Going through the [service worker course on udemy](https://classroom.udacity.com/courses/ud899) and running `npm install` generates a `package.json` file thereby automatically creating a diff.  May confuse newb users when they run `npm reset --hard` as there is still a diff.
@benjaminhoffman
Copy link
Author

benjaminhoffman commented Jan 22, 2018

@jakearchibald would love to get this merged. Helpful for engineers who are just getting started w/ offline-first courses. :)

@benjaminhoffman
Copy link
Author

@jakearchibald bump :)

normally this isn't that big of a deal but it does create confusion for newb devs who may not understand why there's a diff.

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.

1 participant