Skip to content

Commit 87c1ff7

Browse files
authored
chore(npm): remove lockfile (#911)
This removes our lockfile and our `ci-or-install` script. The lockfile caused headaches, since `npm install` would modify the lockfile. Unfortunately, `ci-or-install` (added in #896) didn't work reliably on appveyor, so removing the lockfile seems easier. We remove the script since it's now obsolete. Fixes #893
1 parent 5a6abb5 commit 87c1ff7

File tree

6 files changed

+4
-8102
lines changed

6 files changed

+4
-8102
lines changed

.npmrc

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
# We can't use lockfiles because npm@5 and npm@6 disagree about the syntax (and
2+
# will modify the lockfile when we run `npm install`).
3+
package-lock=false

.travis.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@ node_js:
88
- 8
99
- 9
1010

11-
install:
12-
- npm run ci-or-install
1311
os:
1412
- linux
1513
- osx

appveyor.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ install:
1515
- set PATH=%APPDATA%\npm;%PATH%
1616
- node --version
1717
- npm --version
18-
- npm run ci-or-install
18+
- npm install
1919

2020
matrix:
2121
fast_finish: false

0 commit comments

Comments
 (0)