Skip to content

[26.0] Don't skip pypi-publish job if check-permissions was skipped#22442

Merged
ahmedhamidawan merged 1 commit intogalaxyproject:release_26.0from
nsoranzo:release_26.0_fix_pypi-publish_job
Apr 9, 2026
Merged

[26.0] Don't skip pypi-publish job if check-permissions was skipped#22442
ahmedhamidawan merged 1 commit intogalaxyproject:release_26.0from
nsoranzo:release_26.0_fix_pypi-publish_job

Conversation

@nsoranzo
Copy link
Copy Markdown
Member

@nsoranzo nsoranzo commented Apr 9, 2026

In GitHub Actions, the default behaviour for any job is to run only if all its direct and indirect dependencies have a status of success.
On a release event, the check-permissions job is skipped because of its if: condition.
The build-python-packages job (which depends on check-permissions) runs anyway because it has an explicit if: condition which overrides the default behaviour.
The pypi-publish job (which depends on build-python-packages) was instead skipped because id had no if: condition and one of its ancestors (check-permissions) was skipped.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

In GitHub Actions, the default behaviour for any job is to run only if
all its direct and indirect dependencies have a status of `success`.
On a `release` event, the `check-permissions` job is skipped because of
its `if:` condition.
The `build-python-packages` job (which depends on `check-permissions`)
runs anyway because it has an explicit `if:` condition which overrides
the default behaviour.
The `pypi-publish` job (which depends on `build-python-packages`) was
instead skipped because id had no `if:` condition and one of its
ancestors (`check-permissions`) was skipped.
@github-project-automation github-project-automation Bot moved this to Needs Review in Galaxy Dev - weeklies Apr 9, 2026
@nsoranzo nsoranzo changed the title Don't skip pypi-publish job if check-permissions was skipped [26.0] Don't skip pypi-publish job if check-permissions was skipped Apr 9, 2026
@ahmedhamidawan ahmedhamidawan added this to the 26.1 milestone Apr 9, 2026
@guerler guerler modified the milestones: 26.1, 26.0 Apr 9, 2026
@github-actions github-actions Bot modified the milestones: 26.0, 26.1 Apr 9, 2026
@ahmedhamidawan ahmedhamidawan added area/deployment github_actions Pull requests that update GitHub Actions code labels Apr 9, 2026
@ahmedhamidawan ahmedhamidawan merged commit 609f43e into galaxyproject:release_26.0 Apr 9, 2026
56 of 57 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in Galaxy Dev - weeklies Apr 9, 2026
@nsoranzo nsoranzo deleted the release_26.0_fix_pypi-publish_job branch April 9, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/deployment github_actions Pull requests that update GitHub Actions code kind/bug

Projects

Development

Successfully merging this pull request may close these issues.

3 participants