Skip to content

Refuse a range the shallow clone's depth does not reach - #47

Merged
closer-claudio merged 1 commit into
mainfrom
claudio/20261030-shallow
Oct 30, 2026
Merged

closer-claudio merged 1 commit into
mainfrom
claudio/20261030-shallow

Conversation

@closer-claudio

Copy link
Copy Markdown

A --range git could not resolve was an error in a whole clone but, in a shallow one, fell back to validating what the clone had, from its oldest commit: a depth of one validated the head commit alone and passed, so the commits a pull request added went unseen. That fallback, kept in #39, is removed: the range asked for is the contract, and an unresolvable one is an error in a shallow clone too. The error names the end the clone does not have and what to fetch (fetch-depth: 0, git fetch --deepen or --unshallow). A shallow clone deep enough for the range runs as before. limen's own CI already fetches full history for this reason.

Created by @apostasie personal assistant

A --range git could not resolve was an error in a whole clone but, in a
shallow one, fell back to validating what the clone had, from its oldest
commit: a depth of one validated the head commit alone and passed, so
the commits a pull request added went unseen. The range asked for is
the contract; an unresolvable one is an error in a shallow clone too,
and the error names the end the clone does not have and what to fetch
(fetch-depth: 0, git fetch --deepen or --unshallow). A shallow clone
deep enough for the range runs as before.

Co-Authored-By: Claude Fable 5.1 <[email protected]>
Signed-off-by: closer-claudio <[email protected]>
@closer-claudio

Copy link
Copy Markdown
Author

Reviewed at 283745f

@closer-claudio
closer-claudio merged commit d90eaad into main Oct 30, 2026
12 checks passed
@closer-claudio
closer-claudio deleted the claudio/20261030-shallow branch October 30, 2026 19:04
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.

2 participants