Skip to content
This repository was archived by the owner on Jun 3, 2024. It is now read-only.

Commit fef6a48

Browse files
Fix CI build swallowed errors (e.g. es5 check) (#872)
1 parent a0a2a14 commit fef6a48

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

.circleci/config.yml

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ jobs:
4444
- run:
4545
name: 🔎 Unit Tests
4646
command: |
47+
set -eo pipefail
4748
npm run test-unit
4849
4950
lint-unit-36:
@@ -85,6 +86,7 @@ jobs:
8586
command: |
8687
sudo pip install virtualenv --upgrade
8788
python -m venv venv || virtualenv venv && . venv/bin/activate
89+
set -eo pipefail
8890
pip install --progress-bar off --no-cache-dir -r dev-requirements.txt
8991
- save_cache:
9092
key: dep-{{ checksum ".circleci/config.yml" }}-{{ checksum "ver.txt" }}-{{ checksum "dev-requirements.txt" }}
@@ -94,17 +96,18 @@ jobs:
9496
name: 🏗️ build dash
9597
command: |
9698
. venv/bin/activate && mkdir packages
99+
set -eo pipefail
97100
# build main dash
98101
git clone --depth 1 https://github.com/plotly/dash.git dash-main
99102
cd dash-main && pip install -e .[dev] --progress-bar off && python setup.py sdist && mv dist/* ../packages/
100103
cd dash-renderer && npm ci && npm run build
101104
python setup.py sdist && mv dist/* ../../packages/ && cd ../..
102-
# build dcc
103-
npm ci && npm run build && python setup.py sdist && mv dist/* ./packages
104105
# build html
105106
git clone --depth 1 https://github.com/plotly/dash-html-components.git
106107
cd dash-html-components && npm ci && npm run build
107-
python setup.py sdist && mv dist/* ../packages && cd .. && ls -la packages
108+
python setup.py sdist && mv dist/* ../packages && cd ..
109+
# build dcc
110+
npm ci && npm run build && python setup.py sdist && mv dist/* ./packages && ls -la packages
108111
- persist_to_workspace:
109112
root: ~/project
110113
paths:
@@ -152,6 +155,7 @@ jobs:
152155
name: 🧪 Run Integration Tests
153156
command: |
154157
. venv/bin/activate && rm -rf dash_core_components && ls -la
158+
set -eo pipefail
155159
cd packages && mv dash-*.tar.gz main.tar.gz && ls -la
156160
find . -name "dash_*.gz" | xargs pip install -I --progress-bar off --no-cache-dir
157161
pip install --no-cache-dir --progress-bar off main.tar.gz[dev,testing]
@@ -209,11 +213,11 @@ jobs:
209213
name: 🧪 Run Legacy Integration Tests
210214
command: |
211215
. venv/bin/activate && rm -rf dash_core_components && ls -la
216+
set -eo pipefail
212217
cd packages && mv dash-*.tar.gz main.tar.gz && ls -la
213218
find . -name "dash_*.gz" | xargs pip install -I --progress-bar off --no-cache-dir
214219
pip install --no-cache-dir --progress-bar off main.tar.gz[dev,testing]
215220
pip list | grep dash | xargs pip show && cd ..
216-
set -eo pipefail
217221
echo $(python -V 2>&1) | grep 3. && python -m unittest tests/test_dash_import.py
218222
TESTFILES=$(circleci tests glob "tests/test_integration_*.py" | circleci tests split --split-by=timings)
219223
pytest --durations=10 --junitxml=test-reports/junit_legacy.xml ${TESTFILES}

webpack.config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ module.exports = (env, argv) => {
6565
},
6666
{
6767
test: /\.jsx?$/,
68-
include: /node_modules[\\\/](react-jsx-parser[\\\/])/,
68+
include: /node_modules[\\\/](react-jsx-parser|highlight[.]js)[\\\/]/,
6969
use: {
7070
loader: 'babel-loader',
7171
options: {

0 commit comments

Comments
 (0)