Skip to content

ci: skip the test suite on release-please Release PRs #291

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mogita merged 3 commits into from
Sep 17, 2026
Merged

ci: skip the test suite on release-please Release PRs #291

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
86d4106
ci: skip the test suite on release-please Release PRs
mogita Sep 17, 2026
32fdb0a
ci: verify release-please authorship before skipping checks
mogita Sep 17, 2026
e8b405b
ci: test a human commit pushed onto a Release PR
mogita Sep 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 30 additions & 0 deletions .github/workflows/run_tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,23 @@ env:
permissions:
contents: read

# Every job below skips on a Release PR. release-please only bumps the version and rewrites
# the changelog, and every source commit in one already passed this suite on the PR it came
# from. The guard sits on each job rather than on the calling job in ci.yml, because a job
# skipped by `if:` reports success and satisfies a required status check, while a reusable
# workflow that is never called produces no check at all. It tests the author as well as the
# branch name: on its own, the name would let any PR, a fork's included, call its branch
# release-please--x and skip every required check, which branch protection counts as met.
# github.actor is the third clause, and it is the pusher rather than the PR author, so a
# human commit pushed onto the Release PR to fix a conflict or a changelog entry is tested
# like any other commit instead of riding the skip into main untested.
jobs:
ruff:
if: >-
${{ !(github.actor == 'github-actions[bot]'
&& github.event.pull_request.user.login == 'github-actions[bot]'
&& github.event.pull_request.head.repo.full_name == github.repository
&& startsWith(github.head_ref, 'release-please--')) }}
name: Ruff
runs-on: ubuntu-latest
steps:
Expand All @@ -31,6 +46,11 @@ jobs:
run: make lint

typecheck:
if: >-
${{ !(github.actor == 'github-actions[bot]'
&& github.event.pull_request.user.login == 'github-actions[bot]'
&& github.event.pull_request.head.repo.full_name == github.repository
&& startsWith(github.head_ref, 'release-please--')) }}
name: Type Check (ty)
runs-on: ubuntu-latest
steps:
Expand All @@ -45,6 +65,11 @@ jobs:
# Uses STREAM_CHAT_* credentials which do NOT have video enabled.
# Video paths and manual test paths are defined in the Makefile.
test-non-video:
if: >-
${{ !(github.actor == 'github-actions[bot]'
&& github.event.pull_request.user.login == 'github-actions[bot]'
&& github.event.pull_request.head.repo.full_name == github.repository
&& startsWith(github.head_ref, 'release-please--')) }}
name: Non-video tests (${{ matrix.python-version }})
environment:
name: ci
Expand All @@ -71,6 +96,11 @@ jobs:
# ── Video tests (video-enabled credentials) ─────────────────────
# Uses STREAM_* credentials which have video enabled.
test-video:
if: >-
${{ !(github.actor == 'github-actions[bot]'
&& github.event.pull_request.user.login == 'github-actions[bot]'
&& github.event.pull_request.head.repo.full_name == github.repository
&& startsWith(github.head_ref, 'release-please--')) }}
name: Video tests (${{ matrix.python-version }})
environment:
name: ci
Expand Down