Rename

crates/ruff/src/checkers/ast/analyze/expression.rs

@@ -898,8 +898,8 @@ pub(crate) fn expression(expr: &Expr, checker: &mut Checker) {
             if checker.enabled(Rule::InvalidGetLoggerArgument) {
                 flake8_logging::rules::invalid_get_logger_argument(checker, call);
             }
-            if checker.enabled(Rule::ExcInfoFalseInException) {
-                flake8_logging::rules::exc_info_false_in_exception(checker, call);
+            if checker.enabled(Rule::ExceptionWithoutExcInfo) {
+                flake8_logging::rules::exception_without_exc_info(checker, call);
             }
         }
         Expr::Dict(ast::ExprDict {

crates/ruff/src/codes.rs

@@ -923,7 +923,7 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<(RuleGroup, Rule)> {
         // flake8-logging
         (Flake8Logging, "001") => (RuleGroup::Preview, rules::flake8_logging::rules::DirectLoggerInstantiation),
         (Flake8Logging, "002") => (RuleGroup::Preview, rules::flake8_logging::rules::InvalidGetLoggerArgument),
-        (Flake8Logging, "007") => (RuleGroup::Preview, rules::flake8_logging::rules::ExcInfoFalseInException),
+        (Flake8Logging, "007") => (RuleGroup::Preview, rules::flake8_logging::rules::ExceptionWithoutExcInfo),
         (Flake8Logging, "009") => (RuleGroup::Preview, rules::flake8_logging::rules::UndocumentedWarn),
 
         _ => return None,

crates/ruff/src/rules/flake8_logging/mod.rs

@@ -15,7 +15,7 @@ mod tests {
 
     #[test_case(Rule::DirectLoggerInstantiation, Path::new("LOG001.py"))]
     #[test_case(Rule::InvalidGetLoggerArgument, Path::new("LOG002.py"))]
-    #[test_case(Rule::ExcInfoFalseInException, Path::new("LOG007.py"))]
+    #[test_case(Rule::ExceptionWithoutExcInfo, Path::new("LOG007.py"))]
     #[test_case(Rule::UndocumentedWarn, Path::new("LOG009.py"))]
     fn rules(rule_code: Rule, path: &Path) -> Result<()> {
         let snapshot = format!("{}_{}", rule_code.noqa_code(), path.to_string_lossy());

crates/ruff/src/rules/flake8_logging/rules/exc_info_false_in_exception.rs → crates/ruff/src/rules/flake8_logging/rules/exception_without_exc_info.rs

@@ -31,17 +31,17 @@ use crate::checkers::ast::Checker;
 /// logging.error("...")
 /// ```
 #[violation]
-pub struct ExcInfoFalseInException;
+pub struct ExceptionWithoutExcInfo;
 
-impl Violation for ExcInfoFalseInException {
+impl Violation for ExceptionWithoutExcInfo {
     #[derive_message_formats]
     fn message(&self) -> String {
         format!("Use of `logging.exception` with falsy `exc_info`")
     }
 }
 
 /// LOG007
-pub(crate) fn exc_info_false_in_exception(checker: &mut Checker, call: &ExprCall) {
+pub(crate) fn exception_without_exc_info(checker: &mut Checker, call: &ExprCall) {
     if !is_logger_candidate(
         call.func.as_ref(),
         checker.semantic(),
@@ -60,6 +60,6 @@ pub(crate) fn exc_info_false_in_exception(checker: &mut Checker, call: &ExprCall
     {
         checker
             .diagnostics
-            .push(Diagnostic::new(ExcInfoFalseInException, call.range()));
+            .push(Diagnostic::new(ExceptionWithoutExcInfo, call.range()));
     }
 }

crates/ruff/src/rules/flake8_logging/rules/mod.rs

@@ -1,9 +1,9 @@
 pub(crate) use direct_logger_instantiation::*;
-pub(crate) use exc_info_false_in_exception::*;
+pub(crate) use exception_without_exc_info::*;
 pub(crate) use invalid_get_logger_argument::*;
 pub(crate) use undocumented_warn::*;
 
 mod direct_logger_instantiation;
-mod exc_info_false_in_exception;
+mod exception_without_exc_info;
 mod invalid_get_logger_argument;
 mod undocumented_warn;