Skip to content

Commit 1265946

Browse files
committed
do not call sys.exit(1) for nightly
1 parent a47983e commit 1265946

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

.github/workflows/test.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -54,4 +54,4 @@ jobs:
5454

5555
- if: matrix.python-version == '3.10' && matrix.os == 'ubuntu-latest'
5656
name: Run pytest (against pandas nightly)
57-
run: poetry run poe pytest --nightly ; exit 1 || true
57+
run: poetry run poe pytest --nightly

scripts/_job.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ def __rollback_job(steps: Deque[Step]):
3535
)
3636

3737

38-
def run_job(steps: List[Step]) -> None:
38+
def run_job(steps: List[Step], exit_on_failure: bool = True) -> None:
3939
"""
4040
Responsible to run steps with logs.
4141
"""
@@ -66,5 +66,5 @@ def run_job(steps: List[Step]) -> None:
6666
if not failed:
6767
__rollback_job(rollback_steps)
6868

69-
if failed:
69+
if exit_on_failure and failed:
7070
sys.exit(1)

scripts/test/__init__.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,4 +50,4 @@ def stubtest(allowlist: str, check_missing: bool, nightly: bool) -> None:
5050

5151
def pytest(nightly: bool) -> None:
5252
steps = [_step.nightly] if nightly else []
53-
run_job(steps + [_step.pytest])
53+
run_job(steps + [_step.pytest], exit_on_failure=not nightly)

0 commit comments

Comments
 (0)