Skip to content

Commit 02a4bb8

Browse files
committed
Tests on most OSes
1 parent d6e69c3 commit 02a4bb8

File tree

1 file changed

+30
-26
lines changed

1 file changed

+30
-26
lines changed

.github/workflows/ci.yml

Lines changed: 30 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -42,19 +42,17 @@ jobs:
4242
run: tox -e docs
4343
env:
4444
PYTHON_ISAL_LINK_DYNAMIC: True
45-
test:
46-
runs-on: ubuntu-20.04
45+
test-static:
46+
needs: lint
47+
runs-on: ${{ matrix.os }}
4748
strategy:
4849
matrix:
4950
python-version:
5051
- 3.6
5152
- 3.7
5253
- 3.8
5354
- 3.9
54-
linking_method:
55-
- static
56-
- dynamic
57-
needs: lint
55+
os: ["macos-latest", "ubuntu-latest", "windows-latest"]
5856
steps:
5957
- uses: actions/[email protected]
6058
with:
@@ -65,39 +63,45 @@ jobs:
6563
python-version: ${{ matrix.python-version }}
6664
- name: Install tox and upgrade setuptools and pip
6765
run: pip install --upgrade tox setuptools pip
68-
- name: Install isal
69-
if: ${{ matrix.linking_method == 'dynamic' }}
70-
run: sudo apt-get install libisal-dev
71-
- name: Install yasm # Yasm in pypa/manylinux images.
66+
- name: Install build dependencies (Linux) # Yasm in pypa/manylinux images.
7267
run: sudo apt install yasm
73-
if: ${{ matrix.linking_method == 'static' }}
74-
- name: Run tests (dynamic link)
75-
run: tox -e py3
76-
env:
77-
PYTHON_ISAL_LINK_DYNAMIC: True
78-
if: ${{ matrix.linking_method == 'dynamic' }}
79-
- name: Run tests (dynamic link)
68+
if: ${{ runner.os == "Linux" }}
69+
- name: Install build dependencies (Macos)
70+
run: brew install yasm automake autoconf
71+
if: ${{ runner.os == "MacOS" }}
72+
- name: Install build dependencies (Windows)
73+
run: choco install yasm
74+
if: ${{ runner.os == "Windows" }}
75+
- name: Run tests
8076
run: tox -e py3
81-
if: ${{ matrix.linking_method == 'static' }}
8277
- name: Upload coverage report
83-
if: ${{ matrix.python-version == 3.6 && matrix.linking_method == 'static'}} # Only upload coverage once
8478
uses: codecov/codecov-action@v1
8579

86-
macos:
87-
runs-on: macos-latest
80+
test-dynamic:
81+
runs-on: ubuntu-20.04
8882
needs: lint
83+
strategy:
84+
matrix:
85+
python-version:
86+
- 3.6
87+
- 3.7
88+
- 3.8
89+
- 3.9
8990
steps:
9091
- name: Install python 3.6
9192
uses: actions/[email protected]
9293
with:
9394
submodules: recursive
94-
- name: Set up Python 3.6
95+
- name: Set up Python ${{ matrix.python-version }}
9596
uses: actions/[email protected]
9697
with:
97-
python-version: 3.6
98+
python-version: ${{ matrix.python-version }}
99+
- name: Install isal
100+
run: sudo apt-get install libisal-dev
98101
- name: Install tox and upgrade setuptools and pip
99102
run: pip install --upgrade tox setuptools pip
100-
- name: Install build dependencies
101-
run: brew install yasm automake autoconf
102-
- name: Run tests
103+
104+
- name: Run tests (dynamic link)
103105
run: tox -e py3
106+
env:
107+
PYTHON_ISAL_LINK_DYNAMIC: True

0 commit comments

Comments
 (0)