[test optimization] Update new flaky PR gate documentation#35367
[test optimization] Update new flaky PR gate documentation#35367juan-fernandez merged 4 commits intomasterfrom
Conversation
Preview links (active after the
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3c62e89f0f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
juan-fernandez
force-pushed
the
juan-fernandez/update-new-flaky-pr-gate
branch
from
e5a0235 to
498a7c2
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 220c51e2ca
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| Use this PR Gate to together with [Early Flake Detection][1] to ensure that flakiness is detected on newly added tests. | ||
|
|
||
| [1]: /tests/flaky_tests/early_flake_detection |
There was a problem hiding this comment.
The newly added reference definition reuses [1], which is already used by the document’s existing link map for Static Code Analysis. Markdown reference labels are document-scoped, so this duplicate causes one of the links to resolve to the wrong destination (either the table’s Static Code Analysis link or the new Early Flake Detection link, depending on parser precedence). Switch this new reference to an unused label and update the inline link to avoid silently misrouting readers.
Useful? React with 👍 / 👎.
Co-authored-by: Ursula Chen <58821586+urseberry@users.noreply.github.com>
3 participants
What does this PR do? What is the motivation?
Merge instructions
Merge readiness:
AI assistance
N/A
Additional notes
Preview: https://docs-staging.datadoghq.com/juan-fernandez/update-new-flaky-pr-gate/pr_gates/?tab=newflakytests#rule-types