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

Added package-lock.json for Node.js Tools for Visual Studio #4208

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

Conversation

BertrandMiTek
Copy link

Application of the change:
The change is made in VisualStudio.gitignore for Node.js Tools for Visual Studio

Reasons for making this change:
package-lock.json is automatically generated for any operations where npm modifies either the node_modules tree or package.json. That is why node_modules and package.json must be managed together so that package.json is consistent with what node_modules contains, or else, npm may not work correctly. As it is essential to ignore node_modules, it is also vital to ignore package-lock.json.

Links to documentation supporting these rule changes:
Documentation about package-lock.json: https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json

 * Application of the change: Node.js Tools for Visual Studio
 * Documentation about package-lock.json:
https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json
 * Change made:
package-lock.json is automatically generated for any operations where npm modifies either the node_modules tree, or package.json. That is why node_modules and package.json must be managed together so that package.json is consistent with what node_modules contains, or else, npm may not work properly.
As it is important to ignore node_modules, is also important to ignore package-lock.json.
@cremor
Copy link

cremor commented Apr 27, 2023

The documentation you linked explicitly states "This file is intended to be committed into source repositories".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants