Tidy up some pygrep-hooks rules (#3942)

crates/ruff/src/checkers/physical_lines.rs

@@ -80,9 +80,7 @@ pub fn check_physical_lines(
             }
 
             if enforce_blanket_noqa {
-                if let Some(diagnostic) = blanket_noqa(index, line) {
+                blanket_noqa(&mut diagnostics, index, line);
-                    diagnostics.push(diagnostic);
-                }
             }
 
             if enforce_shebang_missing

crates/ruff/src/rules/pygrep_hooks/rules/blanket_noqa.rs

@@ -19,15 +19,17 @@ impl Violation for BlanketNOQA {
 static BLANKET_NOQA_REGEX: Lazy<Regex> =
     Lazy::new(|| Regex::new(r"(?i)# noqa($|\s|:[^ ])").unwrap());
 
-/// PGH004 - use of blanket noqa comments
+/// PGH004
-pub fn blanket_noqa(lineno: usize, line: &str) -> Option<Diagnostic> {
+pub fn blanket_noqa(diagnostics: &mut Vec<Diagnostic>, lineno: usize, line: &str) {
-    BLANKET_NOQA_REGEX.find(line).map(|m| {
+    if let Some(match_) = BLANKET_NOQA_REGEX.find(line) {
-        Diagnostic::new(
+        let start = line[..match_.start()].chars().count();
+        let end = start + line[match_.start()..match_.end()].chars().count();
+        diagnostics.push(Diagnostic::new(
             BlanketNOQA,
             Range::new(
-                Location::new(lineno + 1, m.start()),
+                Location::new(lineno + 1, start),
-                Location::new(lineno + 1, m.end()),
+                Location::new(lineno + 1, end),
             ),
-        )
+        ));
-    })
+    }
 }

crates/ruff/src/rules/pygrep_hooks/rules/deprecated_log_warn.rs

@@ -16,7 +16,7 @@ impl Violation for DeprecatedLogWarn {
     }
 }
 
-/// PGH002 - deprecated use of logging.warn
+/// PGH002
 pub fn deprecated_log_warn(checker: &mut Checker, func: &Expr) {
     if checker
         .ctx

crates/ruff/src/rules/pygrep_hooks/rules/no_eval.rs

@@ -15,7 +15,8 @@ impl Violation for Eval {
         format!("No builtin `eval()` allowed")
     }
 }
-/// PGH001 - no eval
+
+/// PGH001
 pub fn no_eval(checker: &mut Checker, func: &Expr) {
     let ExprKind::Name { id, .. } = &func.node else {
         return;