Skip to content

Exempt fork parent history from the range gate - #12

Merged
krystophny merged 1 commit into
mainfrom
fork-upstream-commits
Nov 2, 2026
Merged

krystophny merged 1 commit into
mainfrom
fork-upstream-commits

Conversation

@krystophny

Copy link
Copy Markdown
Member

A fork that merges its parent repository inherits commits whose
identities, file sizes and contents it cannot change. In a GitHub fork
the workflow now fetches the parent's branches into refs/upstream/* and
the script excludes commits reachable from them (--upstream-ref).
Identities and secrets are checked on the fork's own commits only. A
(path, blob) pair introduced by a parent commit stays exempt where it
reappears, for example in the fork's merge commit; any fork change to
such a path, and any new path, is checked as before.

On itpplasma/GPEC#10 (develop merged with PrincetonUniversity/GPEC
develop) the gate previously failed on 57.6 MB of parent blobs and three
parent identities; with the parent refs only the merge commit is
checked, and it passes. The job timeout rises to 10 min for the parent
fetch.

Testing: 19/19 unit tests; the patched script run on itpplasma/GPEC#10 with PrincetonUniversity/GPEC heads in refs/upstream/* checks only merge commit 236ab2cd and passes.

Chris&AI


Generated by Claude Code

A fork that merges its parent repository inherits commits whose
identities, file sizes and contents it cannot change. In a GitHub fork
the workflow now fetches the parent's branches into refs/upstream/* and
the script excludes commits reachable from them (--upstream-ref).
Identities and secrets are checked on the fork's own commits only. A
(path, blob) pair introduced by a parent commit stays exempt where it
reappears, for example in the fork's merge commit; any fork change to
such a path, and any new path, is checked as before.

On itpplasma/GPEC#10 (develop merged with PrincetonUniversity/GPEC
develop) the gate previously failed on 57.6 MB of parent blobs and three
parent identities; with the parent refs only the merge commit is
checked, and it passes. The job timeout rises to 10 min for the parent
fetch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01H2kuiWu67fzfh8w4Fkby3P
@krystophny
krystophny merged commit 1ff7888 into main Nov 2, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant