From b9236c4925840d78bec05ea07783422fc7b34fac Mon Sep 17 00:00:00 2001 From: Vinta Chen Date: Sun, 19 Apr 2026 21:55:32 +0800 Subject: [PATCH] refactor(fetch_github_stars): drop unnecessary keyword-only marker on fetch_batch client is the only non-first param and is always required, so the * separator adds no clarity. Update the call site accordingly. Co-Authored-By: Claude --- website/fetch_github_stars.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/website/fetch_github_stars.py b/website/fetch_github_stars.py index bf4666a1..f918ae42 100644 --- a/website/fetch_github_stars.py +++ b/website/fetch_github_stars.py @@ -80,9 +80,7 @@ def parse_graphql_response( return result -def fetch_batch( - repos: list[str], *, client: httpx.Client, -) -> dict[str, dict]: +def fetch_batch(repos: list[str], client: httpx.Client) -> dict[str, dict]: """Fetch star data for a batch of repos via GitHub GraphQL API.""" query = build_graphql_query(repos) if not query: @@ -154,7 +152,7 @@ def main() -> None: print(f"Fetching batch {batch_num}/{total_batches} ({len(batch)} repos)...") try: - results = fetch_batch(batch, client=client) + results = fetch_batch(batch, client) except httpx.HTTPStatusError as e: print(f"HTTP error {e.response.status_code}", file=sys.stderr) if e.response.status_code == 401: