brooktrails/pr-reviewer
Watch
1
0
Fork
You've already forked pr-reviewer
 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

doc(dashboard): say what the call total covers instead of praising it #95

Merged
rcsheets merged 1 commit from feat/inflight-pass-progress into main 2026-08-03 11:19:06 +00:00
Conversation 1 Commits 1 Files changed 1 +1 -1
rcsheets commented 2026-08-03 11:10:18 +00:00
Owner
Copy link

"every pass, counted once" described the feature rather than the number.
The fact a reader needs is what the total spans — the shared pipeline
plus all configs — because the per-column figures deliberately do not add
up to it.

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

"every pass, counted once" described the feature rather than the number. The fact a reader needs is what the total spans — the shared pipeline plus all configs — because the per-column figures deliberately do not add up to it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
doc(dashboard): say what the call total covers instead of praising it
Some checks failed
ci / check (pull_request) Has been cancelled
Details
24a0509735 
"every pass, counted once" described the feature rather than the number.
The fact a reader needs is what the total spans — the shared pipeline
plus all configs — because the per-column figures deliberately do not add
up to it.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
pr-reviewer-bot commented 2026-08-03 11:10:19 +00:00
Collaborator
Copy link

Automated review by pr-reviewer v0.44.0 | Safety Check | Mistral Small | tracking id r-70771b-42f19f
This is an AI-generated review and may contain mistakes.

Status: ✅ Completed

✅ Verdict: LGTM — This is a harmless documentation update that clarifies the meaning of the call total without introducing any issues.
Complexity: trivial

The change replaces the phrase “every pass, counted once” with “the shared pipeline plus every config's review call” to clarify what the call total covers. This is a documentation-only tweak that improves clarity without altering behavior, so no logic errors, security issues, or accidentally committed files are introduced. The diff is minimal and focused, and the new wording accurately describes the scope of the count.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.44.0 | Safety Check | Mistral Small | tracking id `r-70771b-42f19f`* *This is an AI-generated review and may contain mistakes.* **Status:** ✅ Completed --- **✅ Verdict: LGTM** — This is a harmless documentation update that clarifies the meaning of the call total without introducing any issues. **Complexity:** trivial The change replaces the phrase “every pass, counted once” with “the shared pipeline plus every config's review call” to clarify what the call total covers. This is a documentation-only tweak that improves clarity without altering behavior, so no logic errors, security issues, or accidentally committed files are introduced. The diff is minimal and focused, and the new wording accurately describes the scope of the count.
rcsheets force-pushed feat/inflight-pass-progress from 24a0509735
Some checks failed
ci / check (pull_request) Has been cancelled
Details
to 4f04c25f5b
All checks were successful
ci / check (pull_request) Successful in 40s
Details
2026-08-03 11:10:28 +00:00 Compare
rcsheets deleted branch feat/inflight-pass-progress 2026-08-03 11:19:06 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
brooktrails/pr-reviewer!95
No description provided.