Skip to content

Commit cd52bda

Browse files
committed
TST: Move remote data out of matrix
1 parent 0ab85e1 commit cd52bda

File tree

1 file changed

+22
-11
lines changed

1 file changed

+22
-11
lines changed

.github/workflows/ci_tests.yml

Lines changed: 22 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -57,14 +57,6 @@ jobs:
5757
toxenv: py38-test
5858
toxargs: -v
5959

60-
- name: Python 3.8 with all dependencies with remote data
61-
if: github.event_name == 'schedule' && github.repository == 'astropy/astroquery'
62-
os: ubuntu-latest
63-
python: 3.8
64-
toxenv: py38-test-alldeps
65-
toxargs: -v
66-
toxposargs: --remote-data
67-
6860
steps:
6961
- name: Checkout code
7062
uses: actions/checkout@v2
@@ -75,9 +67,7 @@ jobs:
7567
with:
7668
python-version: ${{ matrix.python }}
7769
- name: Install Python dependencies
78-
run: |
79-
python -m pip install --upgrade pip
80-
python -m pip install --upgrade tox
70+
run: python -m pip install --upgrade tox
8171
- name: Run tests
8272
run: tox ${{ matrix.toxargs }} -e ${{ matrix.toxenv }} -- ${{ matrix.toxposargs }}
8373
- name: Upload coverage to codecov
@@ -86,6 +76,27 @@ jobs:
8676
with:
8777
file: ./coverage.xml
8878

79+
# This needs to be outside of matrix because if is not supported
80+
# for only one job in a matrix. But once that is supported, this can go
81+
# back to the matrix above.
82+
remote_data:
83+
name: Python 3.8 with all dependencies with remote data
84+
if: github.event_name == 'schedule' && github.repository == 'astropy/astroquery'
85+
runs-on: ubuntu-latest
86+
steps:
87+
- name: Checkout code
88+
uses: actions/checkout@v2
89+
with:
90+
fetch-depth: 0
91+
- name: Set up Python
92+
uses: actions/setup-python@v2
93+
with:
94+
python-version: '3.8'
95+
- name: Install Python dependencies
96+
run: python -m pip install --upgrade tox
97+
- name: Run tests
98+
run: tox -v -e py38-test-alldeps -- --remote-data
99+
89100
egg_info:
90101
name: egg_info with Python 3.7
91102
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)