From d68491f8996d821d5aedd22e5b2f011e4402bbf9 Mon Sep 17 00:00:00 2001 From: Nicolas Catoni Date: Tue, 28 Jul 2026 17:17:59 +0200 Subject: [PATCH] Adding review rules for codex about adding decorators --- .cursor/rules/pr-review.mdc | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/.cursor/rules/pr-review.mdc b/.cursor/rules/pr-review.mdc index c773de337f4..314aa813689 100644 --- a/.cursor/rules/pr-review.mdc +++ b/.cursor/rules/pr-review.mdc @@ -62,4 +62,11 @@ When adding a new weblog, verify the name is unique across all languages. Search - `bug` and `flaky` markers must include a JIRA ticket (e.g., `bug (JIRA-123)`) - Values with special YAML characters (`>`, `<`, `:`, `#`) must be quoted -- Entries should be sorted alphabetically within each manifest file \ No newline at end of file +- Entries should be sorted alphabetically within each manifest file + +## 11. New decorators + +PRs should not typically create new decorators. If a PR centered around creating, editing, activating or deactivating tests creates new decorators, it is most likely an error and you should comment with the correct approach. +One exception to that would be creating a new scenario. + +- Ref: [skip-tests.md](mdc:docs/edit/skip-tests.md)