Skip to content

Commit a71290d

Browse files
authored
Merge pull request #56 from scientific-python/BUG_fix_test_script
BUG: we should return on the pytest exitval
2 parents 650227b + 2302123 commit a71290d

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

pixi.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ myst-nb = ">=1.1"
4646
jupytext = ">=1.16"
4747
sphinx-book-theme = ">=1.1"
4848
sphinx-copybutton = ">=0.5"
49+
pytest-custom_exit_code = "*"
4950

5051
[feature.py312.dependencies]
5152
python = "3.12.*"

test.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,12 @@ done
2929

3030
pytest --nbval-lax -vv --suppress-no-test-exit-code --durations=10
3131

32+
_exitval="$?"
33+
34+
if [ $_exitval > 0 ]; then
35+
exit $_exitval
36+
fi
37+
3238
# Clean up ipynb files that were converted. Any stray ipynb files that were
3339
# _not_ the result of conversion from Markdown will be left alone.
3440
for file in "${notebook_files[@]}"; do

0 commit comments

Comments
 (0)