From 5cea30d1aefcd06c013eca352a3d29b5170d3493 Mon Sep 17 00:00:00 2001 From: ghinks Date: Fri, 1 May 2026 17:26:07 -0400 Subject: [PATCH] refactor: rename _fetch_single/_detect_single to _fetch_repo/_classify_repo Names now reflect purpose rather than scope: - _fetch_repo mirrors the `fetch` CLI command - _classify_repo mirrors the `classify` CLI command and better reflects that the function computes features, scores outliers, saves results, and returns a RepoClassifyResult Co-Authored-By: Claude Sonnet 4.6 --- src/review_classification/cli/app.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/review_classification/cli/app.py b/src/review_classification/cli/app.py index 0bd738c..d1d64ff 100644 --- a/src/review_classification/cli/app.py +++ b/src/review_classification/cli/app.py @@ -44,7 +44,7 @@ def main( # --------------------------------------------------------------------------- -def _fetch_single( +def _fetch_repo( repo_name: str, start_date: str | None, end_date: str | None, @@ -95,7 +95,7 @@ def _fetch_single( typer.echo(f"Successfully saved {len(prs)} PRs for {repo.owner}/{repo.name}.") -def _detect_single( +def _classify_repo( repo_name: str, threshold: float, min_samples: int, @@ -403,7 +403,7 @@ def fetch( ) for target in targets: - _fetch_single( + _fetch_repo( target.name, target.collate_start, target.collate_end, @@ -516,7 +516,7 @@ def classify( ) repo_results = [ - _detect_single( + _classify_repo( target.name, target.threshold if target.threshold is not None else threshold, target.min_samples if target.min_samples is not None else min_samples,