remove unreachable branch in PGH004 implementation

Author: dylwil3

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

@@ -46,7 +46,6 @@ use crate::Locator;
 #[derive(ViolationMetadata)]
 pub(crate) struct BlanketNOQA {
     missing_colon: bool,
-    space_before_colon: bool,
     file_exemption: bool,
 }
 
@@ -57,27 +56,22 @@ impl Violation for BlanketNOQA {
     fn message(&self) -> String {
         let BlanketNOQA {
             missing_colon,
-            space_before_colon,
             file_exemption,
         } = self;
         // This awkward branching is necessary to ensure that the generic message is picked up by
         // `derive_message_formats`.
-        if !missing_colon && !space_before_colon && !file_exemption {
+        if !missing_colon && !file_exemption {
             "Use specific rule codes when using `noqa`".to_string()
         } else if *file_exemption {
             "Use specific rule codes when using `ruff: noqa`".to_string()
-        } else if *missing_colon {
-            "Use a colon when specifying `noqa` rule codes".to_string()
         } else {
-            "Do not add spaces between `noqa` and its colon".to_string()
+            "Use a colon when specifying `noqa` rule codes".to_string()
         }
     }
 
     fn fix_title(&self) -> Option<String> {
         if self.missing_colon {
             Some("Add missing colon".to_string())
-        } else if self.space_before_colon {
-            Some("Remove space(s) before colon".to_string())
         } else {
             None
         }
@@ -98,7 +92,6 @@ pub(crate) fn blanket_noqa(
                 diagnostics.push(Diagnostic::new(
                     BlanketNOQA {
                         missing_colon: false,
-                        space_before_colon: false,
                         file_exemption: true,
                     },
                     line.range(),
@@ -116,31 +109,14 @@ pub(crate) fn blanket_noqa(
             let mut cursor = Cursor::new(&line[noqa_end.to_usize()..]);
             cursor.eat_while(char::is_whitespace);
 
-            // Check for extraneous spaces before the colon.
-            // Ex) `# noqa : F401`
-            if cursor.first() == ':' {
-                let start = all.end();
-                let end = start + cursor.token_len();
-                let mut diagnostic = Diagnostic::new(
-                    BlanketNOQA {
-                        missing_colon: false,
-                        space_before_colon: true,
-                        file_exemption: false,
-                    },
-                    TextRange::new(all.start(), end),
-                );
-                diagnostic.set_fix(Fix::unsafe_edit(Edit::deletion(start, end)));
-                diagnostics.push(diagnostic);
-            } else if noqa::lex_codes(cursor.chars().as_str()).is_ok_and(|codes| !codes.is_empty())
-            {
+            if noqa::lex_codes(cursor.chars().as_str()).is_ok_and(|codes| !codes.is_empty()) {
                 // Check for a missing colon.
                 // Ex) `# noqa F401`
                 let start = all.end();
                 let end = start + cursor.token_len();
                 let mut diagnostic = Diagnostic::new(
                     BlanketNOQA {
                         missing_colon: true,
-                        space_before_colon: false,
                         file_exemption: false,
                     },
                     TextRange::new(all.start(), end),
@@ -152,7 +128,6 @@ pub(crate) fn blanket_noqa(
                 diagnostics.push(Diagnostic::new(
                     BlanketNOQA {
                         missing_colon: false,
-                        space_before_colon: false,
                         file_exemption: false,
                     },
                     all.range(),