Skip to content

Commit fa493b0

Browse files
raullenchaiclaude
andcommitted
chore(pr_validate): dedupe redundant bad-title patterns
Round 4 of self-validation surfaced one trivial NIT: `^update\.?$` and `^updates?\.?$` are redundant — the `s?` form covers both. Consolidate into the single inclusive pattern with an inline comment so the intent is grep-able. Round 4 verdict: PASS (1 nit, 0 blocking). Tiering converged in 4 rounds: 1→1→0 blocking findings across rounds 1/2/3/4. Without tiering the same 4 rounds would have been 5/5/2/1 "failures" by the old binary verdict — exactly the spiral PR #467 hit. Mission accomplished. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 0692b15 commit fa493b0

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

scripts/pr_validate/steps/cl_description_quality.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,12 +57,11 @@
5757
re.compile(r"^various\s+changes\.?$"),
5858
re.compile(r"^various\s+fixes\.?$"),
5959
re.compile(r"^tweaks?\.?$"),
60-
re.compile(r"^update\.?$"),
60+
re.compile(r"^updates?\.?$"), # covers "update" and "updates"
6161
re.compile(r"^patch\.?$"),
6262
re.compile(r"^wip\.?$"),
6363
re.compile(r"^cleanup\.?$"),
6464
re.compile(r"^changes\.?$"),
65-
re.compile(r"^updates?\.?$"),
6665
re.compile(r"^misc\.?$"),
6766
re.compile(r"^minor\s+(?:fix|change|update)\.?$"),
6867
)

0 commit comments

Comments
 (0)