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

fix(ux): tighten some phrasing #96

Merged
rcsheets merged 2 commits from feat/inflight-pass-progress into main 2026-08-03 11:49:24 +00:00
Conversation 1 Commits 2 Files changed 4 +7 -8
rcsheets commented 2026-08-03 11:35:40 +00:00
Owner
Copy link
No description provided.
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>
doc(dashboard): fix UI copy that names internals or hedges
All checks were successful
ci / check (pull_request) Successful in 40s
Details
03bbb173f6 
Three follow-ups from a sweep of user-facing language:

"step-0 pass" on the review detail page is the pipeline's internal name
for preflight, and told a reader nothing about what produced the
observations they are looking at. It now says which pass and when it
runs.

Two pages said a review is posted to "the MR" while saying "PR"
elsewhere in the same sentence. Forgejo calls them pull requests.

The stalled-review banner made the reader parse a disjunction the page
had already resolved ("a call is still running, or nothing was recorded
— either way..."). It now states the one thing the reader needs: the
review stopped and no result is coming.

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

Automated review by pr-reviewer v0.44.1 | Safety Check | Claude Sonnet 5 | tracking id r-707d0e-c520e2
This is an AI-generated review and may contain mistakes.

Status: ✅ Completed

⚠️ The primary review (Mistral Small) failed. The review below is from Claude Sonnet 5, which reviewed the same diff.

✅ Verdict: LGTM — Purely textual/UX phrasing changes with a matching test update; no functional or security risk.
Complexity: trivial

This is a small, low-risk copy-editing PR: it tightens wording in HTML templates (and updates the corresponding test string) plus fixes leftover "MR" terminology to "PR". No logic changes, no security concerns, no accidental file commits. The test change matches the new template text, so nothing is broken. Nothing alarming here.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.44.1 | Safety Check | Claude Sonnet 5 | tracking id `r-707d0e-c520e2`* *This is an AI-generated review and may contain mistakes.* **Status:** ✅ Completed --- > ⚠️ The primary review (**Mistral Small**) failed. The review below is from **Claude Sonnet 5**, which reviewed the same diff. **✅ Verdict: LGTM** — Purely textual/UX phrasing changes with a matching test update; no functional or security risk. **Complexity:** trivial This is a small, low-risk copy-editing PR: it tightens wording in HTML templates (and updates the corresponding test string) plus fixes leftover "MR" terminology to "PR". No logic changes, no security concerns, no accidental file commits. The test change matches the new template text, so nothing is broken. Nothing alarming here.
rcsheets deleted branch feat/inflight-pass-progress 2026-08-03 11:49:24 +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!96
No description provided.