Update with docs about new flake detection configuration#453
Update with docs about new flake detection configuration#453
Conversation
|
✨ Submitted to Merge by github-actions[bot]. It will be added to the merge queue once all branch protection rules pass and there are no merge conflicts with the target branch. See more details here. |
|
Claude finished @mmatheson's task in 2m 38s —— View job PR Review: Update with docs about new flake detection configuration
SummaryThis PR reorganizes flake detection documentation from a single FindingsMinor issues (non-blocking):
What looks good
No blocking issues found. |
| <!-- SCREENSHOT: Mute button and duration picker on the test case monitor list. | ||
| Show the test case detail page with a monitor's mute button visible, | ||
| and ideally the duration picker dropdown open. --> | ||
|
|
There was a problem hiding this comment.
These `` comment blocks appear throughout the new detection docs as placeholder notes for future screenshots. HTML comments won't render to users, but they add noise to the source. If screenshots will be added before publishing, these are fine to keep temporarily — otherwise they should be removed.
| <!-- SCREENSHOT: Pass-on-Retry monitor configuration panel. | ||
| Show the monitor settings UI with the enabled toggle and recovery days | ||
| slider/input. Capture a state where the monitor is enabled with the | ||
| default 7-day recovery period visible. --> |
There was a problem hiding this comment.
Same screenshot placeholder note as in README.md — consider removing if screenshots won't be added before this is published.
| stale timeout, and branch scope. Capture it with realistic example | ||
| values filled in (e.g., "Main branch flake detector", 30% activation, | ||
| 15% recovery, 6 hour window, 50 min sample, main branch). --> | ||
|
|
There was a problem hiding this comment.
Screenshot placeholder comment — same note as other files. Three of these exist in this file (lines 29–35, 143–147, 209–214). Remove if screenshots won't be added before publishing.
|
/trunk merge |
No description provided.