docs(flaky-tests): document repositories overview landing page#549
docs(flaky-tests): document repositories overview landing page#549samgutentag wants to merge 2 commits intomainfrom
Conversation
Documents the new multi-repo overview dashboard added in trunk-io/trunk2#3527. The landing page at /<org>/flaky-tests now shows all monitored repositories with TESTS, FLAKY, BROKEN counts and a RUNS/DAY sparkline per repo. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Claude finished @samgutentag's task in 58s —— View job Review complete
The new "Repositories overview" section is well-written, clear, and correctly placed before "Key repository metrics" (matching the user flow). Two minor notes left as inline comments:
No typos, grammar issues, or structural problems found. Content is accurate and the section flows naturally into the rest of the page. |
| | **Tests** | Total tracked test cases in the repository | | ||
| | **Flaky** | Number of currently flaky test cases | | ||
| | **Broken** | Number of currently broken test cases | | ||
| | **Runs / Day** | Bar chart of test run volume over the last 10 days, with per-day hover tooltips | |
There was a problem hiding this comment.
The PR description refers to this as a "sparkline" — worth checking which term matches the actual UI. If it's a small inline chart (no axes/labels), "sparkline" is the more precise term.
| | **Runs / Day** | Bar chart of test run volume over the last 10 days, with per-day hover tooltips | | |
| | **Runs / Day** | Sparkline of test run volume over the last 10 days, with per-day hover tooltips | |
|
|
||
| Each repository row displays: | ||
|
|
||
| | Column | Description | |
There was a problem hiding this comment.
The rest of this file uses HTML table syntax (e.g., <table><thead>...) with explicit column widths. This new section uses GFM markdown table syntax. Both render fine in GitBook, but for consistency with the rest of the file you may want to use the HTML format here too. This is a style note, not a blocker.
|
Triage status (2026-05-05): up to date with main. Needs review before merge. |
…verview (#553) ## Summary - Adds a **Repositories overview** section to `flaky-tests/dashboard.md` - Documents the quarantine status icons introduced in trunk-io/trunk2#3538: - Shield icon: quarantining is enabled, auto-quarantine is off - Shield with checkmark icon: auto-quarantine is enabled - Includes full column reference for the repos overview table (Tests, Flaky, Broken, Runs/Day) - Links to the Quarantining page for configuration details ## Source - trunk2 PR: trunk-io/trunk2#3538 ## Notes This PR overlaps with #549 which documents the initial repos overview feature (trunk-io/trunk2#3527). One of the two should be updated to include the other's content before merging, or they can be merged in sequence with a follow-up cleanup. ## Test plan - [ ] Preview in GitBook --------- Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
|
Verification status (2026-05-06): Verified: customers can use this. Ready to publish.
Branch has a merge conflict. Resolve before merge. |
|
Closing as superseded. PR #553 shipped a strict superset of this content to main on 2026-04-something, with additional detail this PR was missing:
The only marginal contribution this PR retained was "per-day hover tooltips" vs main's "per-day tooltips," and tooltips imply hover by default. Not worth a follow-up. Same close-and-skip pattern as #499 and #547. See the verify-docs-pr comment above for the full assessment. |
Summary
flaky-tests/dashboard.mdSource
Test plan