ci: skip test suite for release-* bump PRs#151
Merged
Merged
Conversation
release.py opens the version bump as a PR from a release-<version> branch that only edits pyproject.toml; running the full Docker test suite on it is costly and adds no signal. Gate docker-test off when github.head_ref starts with release- (head_ref is empty for workflow_dispatch, so manual runs still fire; a skipped job counts as success for branch protection).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Skip the
docker-testjob when the PR head branch starts withrelease-.release.pyopens the version bump as a PR from arelease-<version>branch that only editspyproject.toml. Running the full Docker build + test suite (60-min job) on a version-string-only change is costly and adds no signal.How
Extends the existing job-level
if(the fork guard) with&& !startsWith(github.head_ref, 'release-').github.head_refis populated only forpull_requestevents; it's empty forworkflow_dispatch, so manual runs still fire.ifreports as skipped, which counts as success for branch protection — it won't block the bump PR from merging.Targeted, reversible, no behavior change for normal feature PRs.