Azure / ng-deploy-azure

Deploy Angular apps to Azure using the Angular CLI

MIT License

79 stars 37 forks source link

Remove lockfiles #20

Closed bnb closed 5 years ago

bnb commented 5 years ago

Pull Request Template

Description

Removes lockfiles and adds them to .gitignore so they won't be included in the future.

Type of change

Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How to Test

No tests are possible/needed since this is a change that theoretically is invisible.

bnb commented 5 years ago

For context, lockfiles are dangerous for modules since they can provide a false sense of security when working on development like PRs via GitHub but in reality package-lock.json is not published on npm so end-users will actually be getting a differently resolved dependency tree than the one that is resolved in the package-lock.json.