Skip to content

fix(hybrid-gate): diff the docs-only changeset from the merge-base, not the base tip #40

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
forkwright merged 1 commit into from
Aug 15, 2026
Merged

fix(hybrid-gate): diff the docs-only changeset from the merge-base, not the base tip #40

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
10 changes: 9 additions & 1 deletion .github/workflows/hybrid-gate.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -234,8 +234,16 @@ jobs:
# Derive the range from whichever event actually fired. EVENT_BEFORE is all-zeros
# on a branch's first push and may be absent from a shallow fetch, so fall back to
# the head commit alone rather than assuming it resolves.
#
# WHY three-dot on the pull_request arm and two-dot on the push arms: `git diff A..B`
# is a plain comparison of two endpoints, not the range notation it looks like, so
# against a moving base it reports every file the BASE changed since the branch
# forked. A PR touching only README.md is then classified NOT docs-only the moment
# main lands an unrelated .rs commit. `A...B` diffs from the merge-base, which is the
# changeset the PR actually proposes and what the PR's "Files changed" tab shows.
# The push arms are correct as two-dot: before..after IS what that push changed.
if [ -n "$BASE_REF" ]; then
range="origin/${BASE_REF}..HEAD"
range="origin/${BASE_REF}...HEAD"
elif [ -n "$EVENT_BEFORE" ] \
&& git rev-parse --verify --quiet "${EVENT_BEFORE}^{commit}" >/dev/null 2>&1; then
range="${EVENT_BEFORE}..HEAD"
Expand Down