Skip to content

Commit baa752d

Browse files
authored
Merge pull request #6937 from plotly/test-stackgl-bundle
Revise stackgl bundle and add test against undesirable direct changes to `stackgl_modules/index.js`
2 parents 1e6e314 + b605673 commit baa752d

File tree

6 files changed

+11284
-2702
lines changed

6 files changed

+11284
-2702
lines changed

Diff for: .circleci/config.yml

+21
Original file line numberDiff line numberDiff line change
@@ -493,6 +493,25 @@ jobs:
493493
name: Display function constructors in all bundles
494494
command: npm run no-new-func
495495

496+
test-stackgl-bundle:
497+
docker:
498+
- image: cimg/node:16.17.1
499+
working_directory: ~/plotly.js
500+
steps:
501+
- checkout
502+
- run:
503+
name: Set up build environment
504+
command: cd stackgl_modules && npm ci
505+
- run:
506+
name: Bundle bundle-stackgl/index.js
507+
command: cd stackgl_modules && cp index.js INDEX.js && npm run bundle-stackgl
508+
- run:
509+
name: Test stackgl_modules/index.js diff - If failed please remember this file in auto generated and you should not modify it directly until a dependeny change. To suggest changes please submit pull request to the relevant dependency.
510+
command: diff --unified --color stackgl_modules/INDEX.js stackgl_modules/index.js
511+
- store_artifacts:
512+
path: stackgl_modules/index.js
513+
destination: stackgl_modules/index.js
514+
496515
workflows:
497516
version: 2
498517
build-and-test:
@@ -565,3 +584,5 @@ workflows:
565584
- install-and-cibuild
566585

567586
- publish-dist
587+
588+
- test-stackgl-bundle

Diff for: draftlogs/6937_change.md

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
- Regenerate `stackgl_modules/index.js` using updated dependencies [[#6937](https://github.com/plotly/plotly.js/pull/6937)]

0 commit comments

Comments
 (0)