Skip to content

Commit 7e68a2e

Browse files
authored
Merge pull request #3612 from plotly/ci-improve-cache
CircleCI: improve caching for faster startup time
2 parents e8fb19d + 98c962d commit 7e68a2e

File tree

1 file changed

+12
-27
lines changed

1 file changed

+12
-27
lines changed

.circleci/config.yml

Lines changed: 12 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,10 @@ jobs:
1414
working_directory: ~/plotly.js
1515
steps:
1616
- checkout
17-
- restore_cache:
18-
keys:
19-
- v{{ .Environment.CIRCLE_CACHE_VERSION }}-deps-{{ .Branch }}-{{ checksum "package-lock.json" }}
20-
- v{{ .Environment.CIRCLE_CACHE_VERSION }}-deps-master-{{ checksum "package-lock.json" }}
2117
- run:
2218
name: Install dependencies
2319
command: |
24-
npm install
20+
npm ci
2521
- run:
2622
name: List dependency versions
2723
command: |
@@ -33,26 +29,21 @@ jobs:
3329
command: |
3430
npm run pretest
3531
npm run cibuild
36-
- save_cache:
37-
paths:
38-
- node_modules
39-
key: v{{ .Environment.CIRCLE_CACHE_VERSION }}-deps-{{ .Branch }}-{{ checksum "package.json" }}
32+
- run:
33+
command: rm -rf .git
4034
- persist_to_workspace:
41-
root: .
35+
root: /home/circleci
4236
paths:
43-
- node_modules
44-
- build
45-
- dist
37+
- plotly.js
4638

4739
test-jasmine:
4840
docker:
4941
# need '-browsers' version to test in real (xvfb-wrapped) browsers
5042
- image: circleci/node:10.9.0-browsers
5143
working_directory: ~/plotly.js
5244
steps:
53-
- checkout
5445
- attach_workspace:
55-
at: ~/plotly.js
46+
at: ~/
5647
- run:
5748
name: Run jasmine tests (batch 1)
5849
command: ./.circleci/test.sh jasmine
@@ -63,9 +54,8 @@ jobs:
6354
- image: circleci/node:10.9.0-browsers
6455
working_directory: ~/plotly.js
6556
steps:
66-
- checkout
6757
- attach_workspace:
68-
at: ~/plotly.js
58+
at: ~/
6959
- run:
7060
name: Run jasmine tests (batch 2)
7161
command: ./.circleci/test.sh jasmine2
@@ -76,9 +66,8 @@ jobs:
7666
- image: circleci/node:10.9.0-browsers
7767
working_directory: ~/plotly.js
7868
steps:
79-
- checkout
8069
- attach_workspace:
81-
at: ~/plotly.js
70+
at: ~/
8271
- run:
8372
name: Run jasmine tests (batch 3)
8473
command: ./.circleci/test.sh jasmine3
@@ -88,9 +77,8 @@ jobs:
8877
- image: plotly/testbed:latest
8978
working_directory: /var/www/streambed/image_server/plotly.js/
9079
steps:
91-
- checkout
9280
- attach_workspace:
93-
at: /var/www/streambed/image_server/plotly.js/
81+
at: /var/www/streambed/image_server/
9482
- run:
9583
name: Run and setup container
9684
command: |
@@ -108,9 +96,8 @@ jobs:
10896
- image: plotly/testbed:latest
10997
working_directory: /var/www/streambed/image_server/plotly.js/
11098
steps:
111-
- checkout
11299
- attach_workspace:
113-
at: /var/www/streambed/image_server/plotly.js/
100+
at: /var/www/streambed/image_server/
114101
- run:
115102
name: Run and setup container
116103
command: |
@@ -128,9 +115,8 @@ jobs:
128115
- image: circleci/node:10.9.0
129116
working_directory: ~/plotly.js
130117
steps:
131-
- checkout
132118
- attach_workspace:
133-
at: ~/plotly.js
119+
at: ~/
134120
- run:
135121
name: Run syntax tests
136122
command: ./.circleci/test.sh syntax
@@ -140,9 +126,8 @@ jobs:
140126
- image: circleci/node:10.9.0
141127
working_directory: ~/plotly.js
142128
steps:
143-
- checkout
144129
- attach_workspace:
145-
at: ~/plotly.js
130+
at: ~/
146131
- run:
147132
name: Build dist/
148133
command: npm run build

0 commit comments

Comments
 (0)