This commit is contained in:
Daniel Edrisian
2025-09-02 19:46:34 -07:00
parent d3047b27a4
commit cc542eb8df
2 changed files with 21 additions and 4 deletions

View File

@@ -622,12 +622,12 @@ def aggregate_rank(dedup_list: List[Dict[str, Any]], diff_text: str, out_dir: st
def print_progress(passed: int, completed: int, total: int, lock: threading.Lock):
pct = int((passed / total) * 100) if total else 0
pct = int((completed / total) * 100) if total else 0
width = 30
filled = int((passed / total) * width) if total else 0
filled = int((completed / total) * width) if total else 0
bar = "#" * filled + "-" * (width - filled)
with lock:
print(f"[{bar}] {passed}/{total} passed ({pct}%), {completed}/{total} completed")
print(f"[{bar}] {completed}/{total} completed ({pct}%), {passed}/{total} passed")
def print_study_progress(completed: int, total: int, lock: threading.Lock):
@@ -819,14 +819,30 @@ def main():
# Build list of (pr_number, studyguide)
prs = list(data.get("prs") or [])
# Optional: array of PR numbers to ignore when reviewing
ignore_list = data.get("ignore") or []
ignore_set = set()
try:
for x in ignore_list:
try:
ignore_set.add(int(x))
except Exception:
continue
except Exception:
ignore_set = set()
pairs: List[Tuple[int, str]] = []
for p in prs:
n = p.get("number")
sg = p.get("studyguide", "")
if isinstance(n, int) and isinstance(sg, str) and sg.strip():
if n in ignore_set:
continue
pairs.append((n, sg))
if not pairs:
print(f"No studyguides present in reviewers/{reviewer}.json. Try: ./review study {reviewer}", file=sys.stderr)
msg = f"No studyguides present in reviewers/{reviewer}.json. Try: ./review study {reviewer}"
if ignore_set:
msg = f"No studyguides to review after applying ignore list ({len(ignore_set)} ignored). Try: ./review study {reviewer}"
print(msg, file=sys.stderr)
sys.exit(2)
total_available = len(pairs)
if args.limit is not None:

View File

@@ -3,6 +3,7 @@
"reviewer": "bolinfest",
"generated_at": "2025-09-03T02:23:36Z",
"days": 100,
"ignore": [1164],
"prs": [
{
"number": 1164,