Skip to content

Commit

Permalink
Merge pull request #75 from Liam-Deacon/chore/bump-release
Browse files Browse the repository at this point in the history
fix: Always publish pypi packages if any dependent job is skipped
  • Loading branch information
Liam-Deacon authored Jan 21, 2024
2 parents 82ce891 + 3f15202 commit 1366495
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/publish-to-pypi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,7 @@ jobs:
publish_to_testpypi:
name: Publish to TestPyPI
runs-on: ubuntu-latest
if: ${{ github.event_name != 'release' && github.event.inputs.pypi_repo == 'testpypi' }}
if: ${{ always() && (github.event_name != 'release' && github.event.inputs.pypi_repo == 'testpypi') }}
permissions:
id-token: write # IMPORTANT: this permission is mandatory for trusted publishing
needs:
Expand All @@ -246,7 +246,7 @@ jobs:
publish_to_pypi:
name: Publish to PyPI
runs-on: ubuntu-latest
if: ${{ github.event.inputs.pypi_repo == 'pypi' || github.event_name == 'release' }}
if: ${{ always() && (github.event.inputs.pypi_repo == 'pypi' || github.event_name == 'release') }}
permissions:
id-token: write # IMPORTANT: this permission is mandatory for trusted publishing
needs:
Expand All @@ -272,7 +272,7 @@ jobs:
publish_release_artifacts:
name: Upload release artifacts
runs-on: ubuntu-latest
if: ${{ github.event_name == 'release' }}
if: ${{ always() && github.event_name == 'release' }}
permissions:
contents: write
needs:
Expand Down

0 comments on commit 1366495

Please sign in to comment.