From c779cbfa3f932823100e67bf411de1bd6c8a5b69 Mon Sep 17 00:00:00 2001 From: Wei Lee Date: Fri, 24 Jan 2025 18:14:40 +0800 Subject: [PATCH] refactor(AIR302): rename removed_in_3 as removed_expr_in_3 and removed_in_3_function_def as removed_funciton_def_in_3 --- .../ruff_linter/src/checkers/ast/analyze/expression.rs | 10 +++++----- .../ruff_linter/src/checkers/ast/analyze/statement.rs | 2 +- .../src/rules/airflow/rules/removal_in_3.rs | 4 ++-- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/crates/ruff_linter/src/checkers/ast/analyze/expression.rs b/crates/ruff_linter/src/checkers/ast/analyze/expression.rs index 74d877f91b..425432c6aa 100644 --- a/crates/ruff_linter/src/checkers/ast/analyze/expression.rs +++ b/crates/ruff_linter/src/checkers/ast/analyze/expression.rs @@ -176,7 +176,7 @@ pub(crate) fn expression(expr: &Expr, checker: &mut Checker) { pyupgrade::rules::use_pep646_unpack(checker, subscript); } if checker.enabled(Rule::Airflow3Removal) { - airflow::rules::removed_in_3(checker, expr); + airflow::rules::removed_expr_in_3(checker, expr); } pandas_vet::rules::subscript(checker, value, expr); } @@ -227,7 +227,7 @@ pub(crate) fn expression(expr: &Expr, checker: &mut Checker) { refurb::rules::regex_flag_alias(checker, expr); } if checker.enabled(Rule::Airflow3Removal) { - airflow::rules::removed_in_3(checker, expr); + airflow::rules::removed_expr_in_3(checker, expr); } if checker.enabled(Rule::Airflow3MovedToProvider) { airflow::rules::moved_to_provider_in_3(checker, expr); @@ -311,7 +311,7 @@ pub(crate) fn expression(expr: &Expr, checker: &mut Checker) { } } if checker.enabled(Rule::Airflow3Removal) { - airflow::rules::removed_in_3(checker, expr); + airflow::rules::removed_expr_in_3(checker, expr); } if checker.enabled(Rule::MixedCaseVariableInGlobalScope) { if matches!(checker.semantic.current_scope().kind, ScopeKind::Module) { @@ -449,7 +449,7 @@ pub(crate) fn expression(expr: &Expr, checker: &mut Checker) { flake8_pyi::rules::bytestring_attribute(checker, expr); } if checker.enabled(Rule::Airflow3Removal) { - airflow::rules::removed_in_3(checker, expr); + airflow::rules::removed_expr_in_3(checker, expr); } } Expr::Call( @@ -1150,7 +1150,7 @@ pub(crate) fn expression(expr: &Expr, checker: &mut Checker) { ruff::rules::unnecessary_regular_expression(checker, call); } if checker.enabled(Rule::Airflow3Removal) { - airflow::rules::removed_in_3(checker, expr); + airflow::rules::removed_expr_in_3(checker, expr); } if checker.enabled(Rule::UnnecessaryCastToInt) { ruff::rules::unnecessary_cast_to_int(checker, call); diff --git a/crates/ruff_linter/src/checkers/ast/analyze/statement.rs b/crates/ruff_linter/src/checkers/ast/analyze/statement.rs index dc8f69a9c5..56407b2f41 100644 --- a/crates/ruff_linter/src/checkers/ast/analyze/statement.rs +++ b/crates/ruff_linter/src/checkers/ast/analyze/statement.rs @@ -377,7 +377,7 @@ pub(crate) fn statement(stmt: &Stmt, checker: &mut Checker) { flake8_pytest_style::rules::parameter_with_default_argument(checker, function_def); } if checker.enabled(Rule::Airflow3Removal) { - airflow::rules::removed_in_3_function_def(checker, function_def); + airflow::rules::removed_function_def_in_3(checker, function_def); } if checker.enabled(Rule::NonPEP695GenericFunction) { pyupgrade::rules::non_pep695_generic_function(checker, function_def); diff --git a/crates/ruff_linter/src/rules/airflow/rules/removal_in_3.rs b/crates/ruff_linter/src/rules/airflow/rules/removal_in_3.rs index fa028d95b3..2b3b5a7a86 100644 --- a/crates/ruff_linter/src/rules/airflow/rules/removal_in_3.rs +++ b/crates/ruff_linter/src/rules/airflow/rules/removal_in_3.rs @@ -80,7 +80,7 @@ enum Replacement { } /// AIR302 -pub(crate) fn removed_in_3(checker: &mut Checker, expr: &Expr) { +pub(crate) fn removed_expr_in_3(checker: &mut Checker, expr: &Expr) { if !checker.semantic().seen_module(Modules::AIRFLOW) { return; } @@ -117,7 +117,7 @@ pub(crate) fn removed_in_3(checker: &mut Checker, expr: &Expr) { } /// AIR302 -pub(crate) fn removed_in_3_function_def(checker: &mut Checker, function_def: &StmtFunctionDef) { +pub(crate) fn removed_function_def_in_3(checker: &mut Checker, function_def: &StmtFunctionDef) { if !checker.semantic().seen_module(Modules::AIRFLOW) { return; }