Merge pull request #6531 from blueyed/ci-gha-success

ci: GHA: remove unnecessary check for `success()`
This commit is contained in:
Daniel Hahler 2020-01-21 22:38:50 +01:00 committed by GitHub
commit 1cd4bafda7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions

View File

@ -138,18 +138,18 @@ jobs:
run: "tox -e ${{ matrix.tox_env }}" run: "tox -e ${{ matrix.tox_env }}"
- name: Prepare coverage token - name: Prepare coverage token
if: success() && !matrix.skip_coverage && ( github.repository == 'pytest-dev/pytest' || github.event_name == 'pull_request' ) if: (!matrix.skip_coverage && ( github.repository == 'pytest-dev/pytest' || github.event_name == 'pull_request' ))
run: | run: |
python scripts/append_codecov_token.py python scripts/append_codecov_token.py
- name: Combine coverage - name: Combine coverage
if: success() && !matrix.skip_coverage if: (!matrix.skip_coverage)
run: | run: |
python -m coverage combine python -m coverage combine
python -m coverage xml python -m coverage xml
- name: Codecov upload - name: Codecov upload
if: success() && !matrix.skip_coverage if: (!matrix.skip_coverage)
uses: codecov/codecov-action@v1 uses: codecov/codecov-action@v1
with: with:
token: ${{ secrets.codecov }} token: ${{ secrets.codecov }}