mirrors/ruff
1
0
Fork 0
mirror of https://github.com/astral-sh/ruff.git synced 2025-08-04 02:38:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

[airflow] refactor: combine similar case condition (AIR302) (#16944)

Browse source 
<!--
Thank you for contributing to Ruff! To help us out with reviewing,
please consider the following:

- Does this pull request include a summary of the change? (See below.)
- Does this pull request include a descriptive title?
- Does this pull request include references to any relevant issues?
-->

## Summary

<!-- What's the purpose of the change? What does it do, and why? -->

combine similar case condition in AIR302 

## Test Plan

<!-- How was it tested? -->

nothing should be changed. existing test case should already cover it
This commit is contained in:
Wei Lee 2025-03-24 23:36:33 +08:00 • committed by GitHub
parent b442ba440f
commit 581b7005dc
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 3 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

18
crates/ruff_linter/src/rules/airflow/rules/removal_in_3.rs
View file

@ -241,21 +241,9 @@ fn check_call_arguments(checker: &Checker, qualified_name: &QualifiedName, argum
Some("logical_date"),
));
}
["airflow", .., "operators", "datetime", "BranchDateTimeOperator"] => {
checker.report_diagnostics(diagnostic_for_argument(
arguments,
"use_task_execution_day",
Some("use_task_logical_date"),
));
}
["airflow", .., "operators", "weekday", "DayOfWeekSensor"] => {
checker.report_diagnostics(diagnostic_for_argument(
arguments,
"use_task_execution_day",
Some("use_task_logical_date"),
));
}
["airflow", .., "operators", "weekday", "BranchDayOfWeekOperator"] => {
["airflow", .., "operators", "datetime", "BranchDateTimeOperator"]
| ["airflow", .., "operators", "weekday", "DayOfWeekSensor" | "BranchDayOfWeekOperator"] =>
{
checker.report_diagnostics(diagnostic_for_argument(
arguments,
"use_task_execution_day",