Skip to content

Commit 41cc6c8

Browse files
committed
run dist tests in the same container where generated
- run faster as there is no need to persist_to_workspace & remove git - in case of dist failures catched by tests we could simply rerun from failed and do not have to rerun from start
1 parent 54d9c28 commit 41cc6c8

File tree

1 file changed

+0
-17
lines changed

1 file changed

+0
-17
lines changed

.circleci/config.yml

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -187,8 +187,6 @@ jobs:
187187
- run:
188188
name: set up build environment
189189
command: .circleci/env_build.sh
190-
- run:
191-
command: rm -rf .git
192190
- run:
193191
name: Build dist/
194192
command: npm run build
@@ -212,18 +210,6 @@ jobs:
212210
echo https://$CIRCLE_BUILD_NUM-$PROJECT_NUM-gh.circle-artifacts.com/0/dist/plotly.js
213211
echo https://$CIRCLE_BUILD_NUM-$PROJECT_NUM-gh.circle-artifacts.com/0/dist/plotly.min.js
214212
echo https://$CIRCLE_BUILD_NUM-$PROJECT_NUM-gh.circle-artifacts.com/0/dist/plot-schema.json
215-
- persist_to_workspace:
216-
root: ~/
217-
paths:
218-
- plotly.js
219-
220-
test-dist1:
221-
docker:
222-
- image: circleci/node:12.22.1
223-
working_directory: ~/plotly.js
224-
steps:
225-
- attach_workspace:
226-
at: ~/
227213
- run:
228214
name: Test plotly.min.js import using requirejs
229215
command: npm run test-requirejs
@@ -274,6 +260,3 @@ workflows:
274260
- install-and-cibuild
275261

276262
- publish-dist
277-
- test-dist1:
278-
requires:
279-
- publish-dist

0 commit comments

Comments
 (0)