Skip to content

Commit e4a1ea9

Browse files
committed
reverted gh yaml
1 parent 331fb8f commit e4a1ea9

File tree

1 file changed

+31
-13
lines changed

1 file changed

+31
-13
lines changed

.github/workflows/gh-ci.yaml

Lines changed: 31 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -23,20 +23,33 @@ defaults:
2323
shell: bash -l {0}
2424

2525
jobs:
26+
environment-config:
27+
runs-on: ubuntu-latest
28+
outputs:
29+
stable-python-version: ${{ steps.get-compatible-python.outputs.stable-python }}
30+
python-matrix: ${{ steps.get-compatible-python.outputs.python-versions }}
31+
steps:
32+
- uses: actions/setup-python@v5.3.0
33+
with:
34+
python-version: "3.11"
35+
36+
- id: get-compatible-python
37+
uses: MDAnalysis/mdanalysis-compatible-python@main
38+
with:
39+
release: "latest"
40+
2641
main-tests:
2742
if: "github.repository == 'becksteinlab/basicrta'"
43+
needs: environment-config
2844
runs-on: ${{ matrix.os }}
2945
strategy:
3046
fail-fast: false
3147
matrix:
3248
os: [macOS-latest, ubuntu-latest, windows-latest]
33-
python-version: ["3.10", "3.11", "3.12"]
49+
python-version: ${{ fromJSON(needs.environment-config.outputs.python-matrix) }}
3450
mdanalysis-version: ["latest", "develop"]
35-
# Manually exclude any combinations of the test matrix that can't be run
3651
exclude:
37-
# The latest release of MDANalysis only supports up to Python 3.11
38-
# so we exclude 3.12 from the test matrix (issue #20)
39-
- python-version: "3.12"
52+
- python-version: "3.9"
4053
mdanalysis-version: "latest"
4154

4255
steps:
@@ -48,15 +61,17 @@ jobs:
4861
df -h
4962
ulimit -a
5063
64+
5165
# More info on options: https://github.com/conda-incubator/setup-miniconda
5266
- name: Install conda dependencies
5367
uses: conda-incubator/setup-miniconda@v3
5468
with:
5569
python-version: ${{ matrix.python-version }}
5670
environment-file: devtools/conda-envs/test_env.yaml
5771
add-pip-as-python-dependency: true
72+
#architecture: x64
5873

59-
miniforge-variant: Mambaforge
74+
miniforge-variant: Miniforge3
6075
use-mamba: true
6176
channels: conda-forge, defaults
6277

@@ -65,6 +80,7 @@ jobs:
6580
auto-activate-base: false
6681
show-channel-urls: true
6782

83+
6884
- name: Install MDAnalysis version
6985
uses: MDAnalysis/install-mdanalysis@main
7086
with:
@@ -87,22 +103,23 @@ jobs:
87103
conda info
88104
conda list
89105
106+
90107
- name: Run tests
91108
run: |
92-
pytest -n auto -v --cov=basicrta --cov-report=xml --color=yes basicrta/tests/
109+
pytest -n 2 -v --cov=basicrta --cov-report=xml --color=yes basicrta/tests/
93110
94111
- name: codecov
95112
if: github.repository == 'becksteinlab/basicrta' && github.event_name != 'schedule'
96-
uses: codecov/codecov-action@v4
113+
uses: codecov/codecov-action@v3
97114
with:
98115
file: coverage.xml
99116
name: codecov-${{ matrix.os }}-py${{ matrix.python-version }}
100-
token: ${{ secrets.CODECOV_TOKEN }}
101117
verbose: True
102118

103119

104120
pylint_check:
105121
if: "github.repository == 'becksteinlab/basicrta'"
122+
needs: environment-config
106123
runs-on: ubuntu-latest
107124

108125
steps:
@@ -111,7 +128,7 @@ jobs:
111128
- name: Set up Python
112129
uses: actions/setup-python@v4
113130
with:
114-
python-version: "3.11"
131+
python-version: ${{ needs.environment-config.outputs.stable-python-version }}
115132

116133
- name: Install Pylint
117134
run: |
@@ -123,20 +140,21 @@ jobs:
123140
env:
124141
PYLINTRC: .pylintrc
125142
run: |
126-
pylint basicrta
143+
pylint basicrta --exit-zero
127144
128145
129146
pypi_check:
130147
if: "github.repository == 'becksteinlab/basicrta'"
148+
needs: environment-config
131149
runs-on: ubuntu-latest
132150

133151
steps:
134152
- uses: actions/checkout@v4
135153

136-
- name: Set up Python
154+
- name: Set up Python ${{ needs.environment-config.outputs.stable-python-version }}
137155
uses: actions/setup-python@v4
138156
with:
139-
python-version: "3.11"
157+
python-version: ${{ needs.environment-config.outputs.stable-python-version }}
140158

141159
- name: Install dependencies
142160
run: |

0 commit comments

Comments
 (0)