Skip to content

Commit e10c892

Browse files
authored
updated CI workflow to disable windows (#668)
1 parent d112eed commit e10c892

2 files changed

Lines changed: 32 additions & 12 deletions

File tree

.github/workflows/ci.yml

Lines changed: 31 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -16,16 +16,29 @@ jobs:
1616
matrix:
1717
os:
1818
- ubuntu-latest
19-
- windows-latest
19+
# - windows-latest # Temporarily disabled
2020
python-version:
2121
- '3.11'
2222
- '3.13'
23+
test-type:
24+
- unit
25+
- regression
26+
- integration
2327
optional:
2428
- true
2529
include:
2630
- python-version: '3.11'
2731
optional: false
2832
os: ubuntu-latest
33+
test-type: unit
34+
- python-version: '3.11'
35+
optional: false
36+
os: ubuntu-latest
37+
test-type: regression
38+
- python-version: '3.11'
39+
optional: false
40+
os: ubuntu-latest
41+
test-type: integration
2942
runs-on: ${{ matrix.os }}
3043
steps:
3144
- uses: actions/checkout@v6
@@ -54,20 +67,26 @@ jobs:
5467
- name: Create env file
5568
run: |
5669
touch .env
57-
- name: Run unit tests
58-
shell: bash -el {0}
59-
run: pytest . -m unit --cov=h2integrate --cov-report=lcov:./unit-coverage.lcov
60-
- name: Run regression tests
70+
- name: Run ${{ matrix.test-type }} tests
6171
shell: bash -el {0}
62-
run: pytest . -m regression --cov=h2integrate --cov-report=lcov:./regression-coverage.lcov
63-
- name: Run integration tests
64-
shell: bash -el {0}
65-
run: pytest . -m integration --cov=h2integrate --cov-report=lcov:./integration-coverage.lcov
72+
run: pytest . -m ${{ matrix.test-type }} --cov=h2integrate --cov-report=lcov:./${{ matrix.test-type }}-coverage.lcov
6673
- name: Upload test coverage
6774
uses: coverallsapp/github-action@v2
68-
if: contains( matrix.os, 'ubuntu') && contains( matrix.python-version, '3.13')
75+
if: contains(matrix.os, 'ubuntu') && contains(matrix.python-version, '3.13')
76+
with:
77+
github-token: ${{ secrets.COVERALLS_REPO_TOKEN }}
78+
files: ./${{ matrix.test-type }}-coverage.lcov
79+
flag-name: ${{ matrix.test-type }}
80+
parallel: true
81+
fail-on-error: false
82+
coverage-finish:
83+
needs: build
84+
if: always()
85+
runs-on: ubuntu-latest
86+
steps:
87+
- name: Coveralls finished
88+
uses: coverallsapp/github-action@v2
6989
with:
7090
github-token: ${{ secrets.COVERALLS_REPO_TOKEN }}
71-
files: ./unit-coverage.lcov, ./regression-coverage.lcov, ./integration-coverage.lcov
72-
flag-name: unit,regression,integration
91+
parallel-finished: true
7392
fail-on-error: false

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
data between functions in a module. [PR 590](https://github.com/NatLabRockies/H2Integrate/pull/590)
4545
- Adds `H2IntegrateModel.state` as an `IntEnum` to handle setup and run status checks.
4646
[PR 590](https://github.com/NatLabRockies/H2Integrate/pull/590)
47+
- Modified CI setup so Windows is temporarily disabled and also so unit, regression, and integration tests are run in separate jobs to speed up testing and provide more information on test failures. [PR 668](https://github.com/NatLabRockies/H2Integrate/pull/668)
4748

4849
## 0.7.2 [April 9, 2026]
4950

0 commit comments

Comments
 (0)