Skip to content

Commit 3905fab

Browse files
committed
Do not test on Windows due to conda-forge failed release
conda-forge/pytensor-suite-feedstock#177
1 parent 4f0b0ef commit 3905fab

File tree

2 files changed

+9
-8
lines changed

2 files changed

+9
-8
lines changed

.github/workflows/tests.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -178,12 +178,12 @@ jobs:
178178
name: ${{ matrix.os }} ${{ matrix.floatx }}
179179
fail_ci_if_error: false
180180

181-
windows:
181+
should_have_been_windows:
182182
needs: changes
183183
if: ${{ needs.changes.outputs.changes == 'true' }}
184184
strategy:
185185
matrix:
186-
os: [windows-latest]
186+
os: [ubuntu-latest] # should revert back to windows-latest
187187
floatx: [float64]
188188
python-version: ["3.11"]
189189
test-subset:
@@ -206,7 +206,7 @@ jobs:
206206
persist-credentials: false
207207
- uses: mamba-org/setup-micromamba@v2
208208
with:
209-
environment-file: conda-envs/windows-environment-test.yml
209+
environment-file: conda-envs/environment-test.yml
210210
create-args: >-
211211
python=${{matrix.python-version}}
212212
environment-name: pymc-test
@@ -343,7 +343,7 @@ jobs:
343343
if: ${{ needs.changes.outputs.changes == 'true' }}
344344
strategy:
345345
matrix:
346-
os: [windows-latest]
346+
os: [ubuntu-latest] # should revert back ot windows-latest
347347
floatx: [float32]
348348
python-version: ["3.13"]
349349
test-subset:
@@ -362,7 +362,7 @@ jobs:
362362
persist-credentials: false
363363
- uses: mamba-org/setup-micromamba@v2
364364
with:
365-
environment-file: conda-envs/windows-environment-test.yml
365+
environment-file: conda-envs/environment-test.yml
366366
create-args: >-
367367
python=${{matrix.python-version}}
368368
environment-name: pymc-test
@@ -389,12 +389,12 @@ jobs:
389389
all_tests:
390390
if: ${{ always() }}
391391
runs-on: ubuntu-latest
392-
needs: [ changes, ubuntu, windows, macos, alternative_backends, float32 ]
392+
needs: [ changes, ubuntu, macos, alternative_backends, float32 ]
393393
steps:
394394
- name: Check build matrix status
395395
if: ${{ needs.changes.outputs.changes == 'true' &&
396396
( needs.ubuntu.result != 'success' ||
397-
needs.windows.result != 'success' ||
397+
needs.should_have_been_windows.result != 'success' ||
398398
needs.macos.result != 'success' ||
399399
needs.alternative_backends.result != 'success' ||
400400
needs.float32.result != 'success' ) }}

scripts/check_all_tests_are_covered.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import itertools
1111
import logging
1212
import os
13+
import warnings
1314

1415
from pathlib import Path
1516

@@ -110,7 +111,7 @@ def from_yaml():
110111
f"{len(ignored_by_all)} tests are completely ignored:\n{ignored_by_all}"
111112
)
112113
if run_multiple_times:
113-
raise AssertionError(
114+
warnings.warn(
114115
f"{len(run_multiple_times)} tests are run multiple times with the same OS and floatX setting:\n{run_multiple_times}"
115116
)
116117
return

0 commit comments

Comments
 (0)