Skip to content

Update Flaky Tests Management notifications documentation#35763

Merged
TovRudyy merged 7 commits intomasterfrom
olek.rudyy/update-flaky-tests-notifications-docs
Apr 8, 2026
Merged

Update Flaky Tests Management notifications documentation#35763
TovRudyy merged 7 commits intomasterfrom
olek.rudyy/update-flaky-tests-notifications-docs

Conversation

@TovRudyy
Copy link
Copy Markdown
Contributor

@TovRudyy TovRudyy commented Apr 6, 2026

What does this PR do? What is the motivation?

Updates the Flaky Tests Management documentation:

  • Document the new "New flaky test detected" notification type
  • Add a notification types table summarizing all notification types
  • Clarify that code owners matching uses AND logic
  • Restructure the "Receive notifications" section intro for clarity
  • Update bundling language
  • Replace "status" with "state" when referring to flaky test states throughout the page

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

🤖 Generated with Claude Code

TovRudyy and others added 2 commits April 6, 2026 12:05
Add notification types table and document the new "New flaky test detected"
notification type. Restructure the Receive notifications section for clarity.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@TovRudyy TovRudyy marked this pull request as ready for review April 6, 2026 10:08
@TovRudyy TovRudyy requested review from a team as code owners April 6, 2026 10:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

TovRudyy and others added 4 commits April 6, 2026 14:27
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
| **New flaky test detected** | A new flaky test is detected on the default branch of the repository. |
| **Test quarantined** | A test is quarantined by an automated policy rule (time-based, branch-based, or failure rate). |
| **Test disabled** | A test is disabled by an automated policy rule (time-based, branch-based, or failure rate). |
| **Fix successful** | A test passes all retries in the remediation flow and is marked as fix in progress. |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it supposed to be "In progress" or "Fix in progress. Is this separate from ln 128?
If all retries pass, marks the fix as **in progress** in the Flaky Tests Management UI,

Suggested change
| **Fix successful** | A test passes all retries in the remediation flow and is marked as fix in progress. |
| **Fix successful** | A test passes all retries in the remediation flow and is marked as in progress. |

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will change it to match in progress, thanks

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, fix in progress sounds better. Nevermind

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@TovRudyy TovRudyy merged commit 68895b7 into master Apr 8, 2026
15 checks passed
@TovRudyy TovRudyy deleted the olek.rudyy/update-flaky-tests-notifications-docs branch April 8, 2026 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants