Repository navigation
Cover every fork's reimpl/ branches from the rollup tooling itself - #394
Merged
Merged
Conversation
status_rollup.py gains --fetch-forks (REIMPL_FETCH_FORKS=1): it enumerates the root repository's forks with gh, adds each as a remote and shallow-fetches its reimpl/ branches before scanning, so a local run covers what CI covers. It also reports per-fork branch counts, and says so when nothing but the local repo is fetched. Fork identity comes from the remote's url rather than its name, since an 'owner-name' remote cannot be split unambiguously. status_regression.py understands the same two variables. The touched-file list is now settled before the fetch: a shallow fork fetch grafts the commit under test as parentless when the PR comes from one of those branches, which would turn the normal case into the script's "no merge base" hard error. The three workflows drop their fetch_fork_branches.sh step in favour of that flag, so the fetch and the scan cannot drift apart. The script stays as a standalone utility, with a PowerShell port beside it; both now default to sourcehold/OpenSHC. pick_work.py reports where to get each pick: the URL of the file behind the rollup's best_source, and with --all-sources every fork branch that has the function, collapsed by blob so the dozens of branches cut from one another show as one entry. New --format urls. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
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.
status_rollup.pygains--fetch-forks(orREIMPL_FETCH_FORKS=1): it enumerates the root repository's forks withgh, adds each as a remote and shallow-fetches itsreimpl/branches before scanning, so a local run covers what CI covers. It also reports per-fork branch counts and says so when nothing but the local repo is fetched. Fork identity comes from the remote's url rather than its name, since anowner-nameremote cannot be split unambiguously.status_regression.pyunderstands the same two variables. The touched-file list is now settled before the fetch: a shallow fork fetch grafts the commit under test as parentless when the PR comes from one of those branches, which would turn the normal case into the script's "no merge base" hard error.The three workflows drop their
fetch_fork_branches.shstep in favour of that flag, so the fetch and the scan cannot drift apart. The script stays as a standalone utility, with a PowerShell port beside it; both now default tosourcehold/OpenSHC.pick_work.pyreports where to get each pick: the URL of the file behind the rollup'sbest_source, and with--all-sourcesevery fork branch that has the function, collapsed by blob so the dozens of branches cut from one another show as one entry. New--format urls.Tested locally against the live repository: 9 forks enumerated, 198 branch tips fetched, 3704 rollup rows (unchanged totals); the regression check verified on all four paths (touched-list, no-fetch default, merge-base-before-fetch, and a seeded 3-entry regression exiting 2).
🤖 Generated with Claude Code