Skip to content

Commit 8b14c22

Browse files
author
Shyue Ping Ong
committed
Revert bad changes to the test workflow.
1 parent f7360a8 commit 8b14c22

File tree

1 file changed

+1
-12
lines changed

1 file changed

+1
-12
lines changed

.github/workflows/test.yml

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -118,20 +118,10 @@ jobs:
118118
python -m pip install numpy cython packaging
119119
python -m pip install -e '.[dev,optional]'
120120
- name: pytest split ${{ matrix.split }}
121-
if: github.event_name == 'release' || github.event_name == 'push'
122-
# For releases, we do non-parallel runs, which usually makes sure that the coverage is properly completed.
123121
run: |
124-
pytest --cov=pymatgen tests
125-
- name: pytest split ${{ matrix.split }}
126-
if: github.event_name == 'pull_request'
127-
# To update the test durations, do pip install pytest-split and run
128-
# pytest --store-durations --durations-path tests/files/.pytest-split-durations
129-
# and commit the results (requires pip install pytest-split)
130-
run: |
131-
pytest --splits 10 --group ${{ matrix.split }} --durations-path tests/files/.pytest-split-durations tests
122+
pytest --cov=pymatgen --splits 10 --group ${{ matrix.split }} --durations-path tests/files/.pytest-split-durations tests
132123
- name: Upload coverage
133124
# Only upload coverage for ubuntu py3.11 runs. Continue on error as coverage is not critical.
134-
continue-on-error: true
135125
if: github.event_name != 'pull_request' && matrix.os == 'ubuntu-latest' && matrix.python-version == '3.11'
136126
uses: actions/upload-artifact@v3
137127
with:
@@ -140,7 +130,6 @@ jobs:
140130

141131
coverage:
142132
# Only upload coverage for ubuntu py3.11 runs. Continue on error as coverage is not critical.
143-
continue-on-error: true
144133
if: github.event_name != 'pull_request'
145134
needs: test
146135
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)