Skip to content

Commit 1935824

Browse files
authored
devops: add package-lock.json (#1859) (#1889)
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.
1 parent c2fe55e commit 1935824

File tree

8 files changed

+5784
-38
lines changed

8 files changed

+5784
-38
lines changed

.appveyor.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ build: off
66

77
install:
88
- ps: Install-Product node $env:nodejs_version
9-
- npm install
9+
- npm ci
1010
- npm run ctest
1111
- npm run wtest
1212
- npm run ftest

.circleci/config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ jobs:
1010
key: dependency-cache-{{ checksum "package.json" }}
1111

1212
- run:
13-
command: npm install
13+
command: npm ci
1414

1515
- save_cache:
1616
key: dependency-cache-{{ checksum "package.json" }}

.github/workflows/infra.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,5 +25,5 @@ jobs:
2525
sudo apt-get update
2626
sudo apt-get install libgbm-dev
2727
sudo apt-get install xvfb
28-
- run: npm install
28+
- run: npm ci
2929
- run: npm run lint

.github/workflows/rebase.yml

Lines changed: 0 additions & 24 deletions
This file was deleted.

.github/workflows/tests.yml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
- name: install xvfb
2727
run: |
2828
sudo apt-get install xvfb
29-
- run: npm install
29+
- run: npm ci
3030
# XVFB-RUN merges both STDOUT and STDERR, whereas we need only STDERR
3131
# Wrap `npm run` in a subshell to redirect STDERR to file.
3232
- run: xvfb-run --auto-servernum -- bash -c "npm run ccoverage 2>./chromium-linux-testrun.log"
@@ -55,7 +55,7 @@ jobs:
5555
with:
5656
node-version: 10
5757
- uses: microsoft/playwright-github-action@v1
58-
- run: npm install
58+
- run: npm ci
5959
- run: npm run ctest 2>./chromium-mac-testrun.log
6060
env:
6161
DEBUG: "*"
@@ -82,7 +82,7 @@ jobs:
8282
with:
8383
node-version: 10
8484
- uses: microsoft/playwright-github-action@v1
85-
- run: npm install
85+
- run: npm ci
8686
- run: npm run ctest 2>./chromium-win-testrun.log
8787
shell: bash
8888
env:
@@ -113,7 +113,7 @@ jobs:
113113
- name: install xvfb
114114
run: |
115115
sudo apt-get install xvfb
116-
- run: npm install
116+
- run: npm ci
117117
# XVFB-RUN merges both STDOUT and STDERR, whereas we need only STDERR
118118
# Wrap `npm run` in a subshell to redirect STDERR to file.
119119
- run: xvfb-run --auto-servernum -- bash -c "npm run wcoverage 2>./webkit-linux-testrun.log"
@@ -142,7 +142,7 @@ jobs:
142142
with:
143143
node-version: 10
144144
- uses: microsoft/playwright-github-action@v1
145-
- run: npm install
145+
- run: npm ci
146146
- run: npm run wtest 2>./webkit-mac-testrun.log
147147
env:
148148
DEBUG: "*,-pw:wrapped*"
@@ -169,7 +169,7 @@ jobs:
169169
with:
170170
node-version: 10
171171
- uses: microsoft/playwright-github-action@v1
172-
- run: npm install
172+
- run: npm ci
173173
- run: npm run wtest 2>./webkit-win-testrun.log
174174
shell: bash
175175
env:
@@ -200,7 +200,7 @@ jobs:
200200
- name: install xvfb
201201
run: |
202202
sudo apt-get install xvfb
203-
- run: npm install
203+
- run: npm ci
204204
# XVFB-RUN merges both STDOUT and STDERR, whereas we need only STDERR
205205
# Wrap `npm run` in a subshell to redirect STDERR to file.
206206
- run: xvfb-run --auto-servernum -- bash -c "npm run fcoverage 2>./firefox-linux-testrun.log"
@@ -229,7 +229,7 @@ jobs:
229229
with:
230230
node-version: 10
231231
- uses: microsoft/playwright-github-action@v1
232-
- run: npm install
232+
- run: npm ci
233233
- run: npm run ftest 2>./firefox-mac-testrun.log
234234
env:
235235
DEBUG: "*"
@@ -256,7 +256,7 @@ jobs:
256256
with:
257257
node-version: 10
258258
- uses: microsoft/playwright-github-action@v1
259-
- run: npm install
259+
- run: npm ci
260260
- run: npm run ftest 2>./firefox-win-testrun.log
261261
shell: bash
262262
env:
@@ -285,5 +285,5 @@ jobs:
285285
# we use in `installation-tests.sh`.
286286
node-version: 12
287287
- uses: microsoft/playwright-github-action@v1
288-
- run: npm install
288+
- run: npm ci
289289
- run: bash test/installation-tests/installation-tests.sh

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
*.swp
1010
*.pyc
1111
.vscode
12-
package-lock.json
1312
yarn.lock
1413
/node6
1514
/src/generated/*

.travis.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,8 @@ addons:
2626
- xvfb
2727
notifications:
2828
email: false
29+
install:
30+
- npm ci
2931
cache:
3032
directories:
3133
- node_modules

0 commit comments

Comments
 (0)