From 20a3d5a77430279535ff9d1570d79008a87b715e Mon Sep 17 00:00:00 2001 From: David Karlsson <35727626+dvdksn@users.noreply.github.com> Date: Tue, 2 Jun 2026 12:36:47 +0200 Subject: [PATCH] Fix bot review false positives: drop fabricated 80-char rule, clarify enable/disable The PR-review bot's prompt instructed it to flag lines over 80 characters, but no such rule exists in the repo (markdownlint MD013 is disabled and STYLE.md never mentions it). Remove that instruction from pr-review.yml. Also clarify the STYLE.md word list so "turn on/off" applies to UI toggles while "enable/disable" stays acceptable in general prose, matching STYLE.md's own example. This stops the bot from flagging every "enable" as a violation. Co-Authored-By: Claude Opus 4.8 (1M context) --- .github/workflows/pr-review.yml | 1 - STYLE.md | 6 ++++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/.github/workflows/pr-review.yml b/.github/workflows/pr-review.yml index bf389e1ef5..60d16270b1 100644 --- a/.github/workflows/pr-review.yml +++ b/.github/workflows/pr-review.yml @@ -50,7 +50,6 @@ jobs: ### 3. Markdown Formatting - Poor markdown syntax (unclosed code blocks, broken lists, indentation issues, etc.) - - Line wrapping over 80 characters (except links, code blocks, tables) ### 4. AI-Generated Patterns (HIGH PRIORITY) Flag AI-isms from STYLE.md: diff --git a/STYLE.md b/STYLE.md index e5c9a86914..aca8d1477e 100644 --- a/STYLE.md +++ b/STYLE.md @@ -464,6 +464,12 @@ _1. Optional. Enter a description for the job._ | use | utilize | | help | facilitate | +Use "turn on" and "turn off" for UI toggles, switches, and settings a user +flips in the interface. In general prose, "enable" and "disable" are +acceptable when describing a feature or capability becoming available — for +example, "When you enable auto-log-out, your users are logged out." Don't +treat every "enable" or "disable" as a violation. + ### Version numbers - Use "earlier" not "lower": Docker Desktop 4.1 and earlier