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

devops: add package-lock.json #1859

Merged
merged 1 commit into from
Apr 19, 2020

Conversation

aslushnikov
Copy link
Contributor

We need to commit package-lock.json to repository to ensure
reproducible builds.

This also starts using npm ci to
install dependencies.

Drive-by: remove non-functioning rebase workflow.

We need to commit `package-lock.json` to repository to ensure
reproducible builds.

This also starts using [`npm ci`](https://docs.npmjs.com/cli/ci.html) to
install dependencies.

Drive-by: remove non-functioning `rebase` workflow.
@aslushnikov aslushnikov merged commit 75f35e4 into microsoft:master Apr 19, 2020
@aslushnikov aslushnikov deleted the add-package-lock branch April 19, 2020 05:14
pavelfeldman added a commit that referenced this pull request Apr 19, 2020
This reverts commit 75f35e4.

It is unclear how to add/remove dependencies now.
aslushnikov added a commit to aslushnikov/playwright that referenced this pull request Apr 20, 2020
We need to commit `package-lock.json` to repository to ensure
reproducible builds.

This also starts using [`npm ci`](https://docs.npmjs.com/cli/ci.html) to
install dependencies.

Drive-by: remove non-functioning `rebase` workflow.
aslushnikov added a commit that referenced this pull request Apr 20, 2020
We need to commit `package-lock.json` to repository to ensure
reproducible builds.

This also starts using [`npm ci`](https://docs.npmjs.com/cli/ci.html) to
install dependencies.

Drive-by: remove non-functioning `rebase` workflow.
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.

2 participants