Merge pull request #219 from edx/efischer/use_ci

fix(ci): use npm ci on travis

.travis.yml

@@ -1,13 +1,16 @@
 language: node_js
 node_js:
 - 8.11
+install:
+  - npm i -g [email protected]
+  - npm ci
 branches:
   only:
   - master
   - "/v\\d+\\.\\d+\\.d+$/"
 cache:
   directories:
-  - node_modules
+  - "$HOME/.npm"
 stages:
 - PR checks ✅
 - name: "Ship It \U0001F6A2"

.travis/validate_changes_for_merge.sh

@@ -4,7 +4,6 @@
 
 EXIT_STATUS=0
 npm run lint || EXIT_STATUS=$?
-make package-lock.validate || EXIT_STATUS=$?
 make i18n.pre_validate || EXIT_STATUS=$?
 commitlint-travis || EXIT_STATUS=$?
 exit $EXIT_STATUS

Dockerfile

@@ -3,6 +3,7 @@ FROM node:8.11
 USER root
 RUN apt-get update
 RUN apt-get install -y vim
+RUN npm i -g [email protected]
 
 WORKDIR /prebuilt
 COPY package.json .

Makefile

@@ -78,8 +78,5 @@ pull_translations: ## must be exactly this name for edx tooling support, see eco
 	# explicit list of languages defined here and in currentlySupportedLangs.jsx
 	tx pull -f --mode reviewed --language="ar,fr,es_419,zh_CN"
 
-package-lock.validate:
-	git diff --name-only --exit-code package-lock.json
-
 copy-dist:
 	for f in dist/*; do docker cp $$f edx.devstack.studio:/edx/app/edxapp/edx-platform/node_modules/@edx/studio-frontend/dist/; done