From b90c4869f9d013da5ec5852ab308266d758a629e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Miguel=20Raz=20Guzm=C3=A1n=20Macedo?= Date: Thu, 27 Aug 2026 05:53:18 +0000 Subject: [PATCH] Require clear sections in lint UI fixtures (#62659) Add `// --- SHOULD FIRE ---- ` and `// ---- SHOULD NOT FIRE ---- ` to the `/lint-creator` skill. Release Notes: - N/A --- .agents/skills/lint-creator/SKILL.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/.agents/skills/lint-creator/SKILL.md b/.agents/skills/lint-creator/SKILL.md index 0fd81a54a5..a1861dc50c 100644 --- a/.agents/skills/lint-creator/SKILL.md +++ b/.agents/skills/lint-creator/SKILL.md @@ -46,3 +46,12 @@ description: Add, modify, or test a custom dylint lint in `tooling/lints`. Cover ``` (defaults to `--workspace` if no package is given; the script handles the `--force-warn` and cache-cleaning gotchas documented in the README). + +## UI fixture sections + +Every lint-specific `ui/*.rs` fixture MUST put cases expected to trigger the lint first, followed by cases expected not to trigger it, using exactly these headings: + +```rust +// ==================== SHOULD FIRE ==================== +// ================== SHOULD NOT FIRE ================== +```