Skip to content

refactor: extract fetch_all_repos helper to deduplicate pagination - #30

Merged
sachiniyer merged 1 commit into
mainfrom
siyer/dedup-pagination
Feb 13, 2026
Merged

sachiniyer merged 1 commit into
mainfrom
siyer/dedup-pagination

Conversation

@sachiniyer

Copy link
Copy Markdown
Contributor

Both branches of resolve_repo_id() had nearly identical pagination
loops. Extract a shared fetch_all_repos() helper and filter the
results inline.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

@sachiniyer
sachiniyer marked this pull request as ready for review February 13, 2026 08:21
@sachiniyer
sachiniyer force-pushed the siyer/pagination-constant branch 2 times, most recently from 5b24a5e to ea83632 Compare February 13, 2026 08:26
@sachiniyer
sachiniyer force-pushed the siyer/dedup-pagination branch from b57e4d2 to 014f060 Compare February 13, 2026 08:27
@sachiniyer
sachiniyer force-pushed the siyer/pagination-constant branch from ea83632 to 5aedebc Compare February 13, 2026 08:29
@sachiniyer
sachiniyer force-pushed the siyer/dedup-pagination branch from 014f060 to 0b17b6b Compare February 13, 2026 08:29
@sachiniyer
sachiniyer changed the base branch from siyer/pagination-constant to graphite-base/30 February 13, 2026 08:33
@sachiniyer
sachiniyer force-pushed the siyer/dedup-pagination branch from 0b17b6b to 5ccf29c Compare February 13, 2026 08:33
@graphite-app
graphite-app Bot changed the base branch from graphite-base/30 to main February 13, 2026 08:34
@graphite-app

graphite-app Bot commented Feb 13, 2026 •

Copy link
Copy Markdown

Merge activity

  • Feb 13, 8:34 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.
  • Feb 13, 8:40 AM UTC: @sachiniyer merged this pull request with Graphite.

@sachiniyer
sachiniyer force-pushed the siyer/dedup-pagination branch from 5ccf29c to 3a80ea3 Compare February 13, 2026 08:34
Both branches of resolve_repo_id() had nearly identical pagination
loops. Extract a shared fetch_all_repos() helper and filter the
results inline.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sachiniyer
sachiniyer force-pushed the siyer/dedup-pagination branch from 3a80ea3 to 359f5f9 Compare February 13, 2026 08:36
@sachiniyer
sachiniyer merged commit 2cc2782 into main Feb 13, 2026
11 checks passed
@sachiniyer
sachiniyer deleted the siyer/dedup-pagination branch February 13, 2026 08:41
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