refactor(linter): remove static lifetime from disable directives function argument (#13492)

camc314 · camc314 · commit e0396fdacd86 · 2025-08-31T18:43:15.000Z
diff --git a/crates/oxc_linter/src/disable_directives.rs b/crates/oxc_linter/src/disable_directives.rs
@@ -125,7 +125,7 @@ impl<'a> DisableDirectives<'a> {
         self.used_disable_comments.borrow_mut().push(disable_directive);
     }
 
-    pub fn contains(&self, rule_name: &'static str, span: Span) -> bool {
+    pub fn contains(&self, rule_name: &str, span: Span) -> bool {
         // For `eslint-disable-next-line` and `eslint-disable-line` directives, we only check
         // if the diagnostic's starting position falls within the disabled interval.
         // This prevents suppressing diagnostics for larger constructs (like functions) that

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ impl<'a> DisableDirectives<'a> {`
`125`	`125`	`self.used_disable_comments.borrow_mut().push(disable_directive);`
`126`	`126`	`}`
`127`	`127`
`128`		`- pub fn contains(&self, rule_name: &'static str, span: Span) -> bool {`
	`128`	`+ pub fn contains(&self, rule_name: &str, span: Span) -> bool {`
`129`	`129`	// For `eslint-disable-next-line` and `eslint-disable-line` directives, we only check
`130`	`130`	`// if the diagnostic's starting position falls within the disabled interval.`
`131`	`131`	`// This prevents suppressing diagnostics for larger constructs (like functions) that`