Skip to content

Commit

Permalink
Use npm ci instead of npm install in CI scripts (#4688)
Browse files Browse the repository at this point in the history
  • Loading branch information
kravets-levko authored Feb 26, 2020
1 parent 9790b07 commit 7043951
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 7 deletions.
6 changes: 3 additions & 3 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ jobs:
steps:
- checkout
- run: mkdir -p /tmp/test-results/eslint
- run: npm install
- run: npm ci
- run: npm run lint:ci
- store_test_results:
path: /tmp/test-results
Expand All @@ -72,7 +72,7 @@ jobs:
- checkout
- run: sudo apt install python3-pip
- run: sudo pip3 install -r requirements_bundles.txt
- run: npm install
- run: npm ci
- run: npm run bundle
- run: npm test
- run: npm run lint
Expand All @@ -91,7 +91,7 @@ jobs:
- run:
name: Install npm dependencies
command: |
npm install
npm ci
- run:
name: Setup Redash server
command: |
Expand Down
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ FROM node:12 as frontend-builder

WORKDIR /frontend
COPY package.json package-lock.json /frontend/
RUN npm install
RUN npm ci

COPY client /frontend/client
COPY webpack.config.js /frontend/
Expand Down Expand Up @@ -39,7 +39,7 @@ RUN apt-get update && \
default-libmysqlclient-dev \
freetds-dev \
libsasl2-dev && \
# MSSQL ODBC Driver:
# MSSQL ODBC Driver:
curl https://packages.microsoft.com/keys/microsoft.asc | apt-key add - && \
curl https://packages.microsoft.com/config/debian/10/prod.list > /etc/apt/sources.list.d/mssql-release.list && \
apt-get update && \
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ backend-unit-tests: up test_db
docker-compose run --rm --name tests server tests

frontend-unit-tests: bundle
npm install
npm ci
npm run bundle
npm test

Expand Down
2 changes: 1 addition & 1 deletion netlify.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
[build]
base = "client"
publish = "client/dist"
command = "npm install && npm run build"
command = "npm ci && npm run build"

[[redirects]]
from = "/api/*"
Expand Down

0 comments on commit 7043951

Please sign in to comment.