Stabilize the remaining Airflow rules by ntBre · Pull Request #20250 · astral-sh/ruff (original) (raw)

@ntBre added this to the v0.13 milestone

Sep 4, 2025

@ntBre ntBre added the rule

Implementing or modifying a lint rule

label

Sep 4, 2025

ntBre

@ntBre ntBre marked this pull request as ready for review

September 4, 2025 19:59

@ntBre ntBre linked an issue

Sep 4, 2025

that may beclosed by this pull request

@ntBre

Minor docs tweaks, but the tests for all of these are in the right location

Restore "removed" language in error and snapshots, see #20250 (comment)

@ntBre

@ntBre

@ntBre

@ntBre

@ntBre ntBre mentioned this pull request

Sep 5, 2025

@ntBre ntBre deleted the brent/airflow branch

September 8, 2025 14:01

@ntBre ntBre mentioned this pull request

Sep 8, 2025

2 tasks

ntBre added a commit that referenced this pull request

Sep 8, 2025

@ntBre

I put this all in one PR to make it easier to double check with @Lee-W before we merge this. I also made a few minor documentation changes and updated one error message that I want to make sure are okay. But for the most part this just moves the rules from RuleGroup::Preview to RuleGroup::Stable!

Fixes #17749

ntBre added a commit that referenced this pull request

Sep 10, 2025

@ntBre

I put this all in one PR to make it easier to double check with @Lee-W before we merge this. I also made a few minor documentation changes and updated one error message that I want to make sure are okay. But for the most part this just moves the rules from RuleGroup::Preview to RuleGroup::Stable!

Fixes #17749

ntBre added a commit that referenced this pull request

Sep 10, 2025

@ntBre

I put this all in one PR to make it easier to double check with @Lee-W before we merge this. I also made a few minor documentation changes and updated one error message that I want to make sure are okay. But for the most part this just moves the rules from RuleGroup::Preview to RuleGroup::Stable!

Fixes #17749

ntBre added a commit that referenced this pull request

Sep 10, 2025

@ntBre

I put this all in one PR to make it easier to double check with @Lee-W before we merge this. I also made a few minor documentation changes and updated one error message that I want to make sure are okay. But for the most part this just moves the rules from RuleGroup::Preview to RuleGroup::Stable!

Fixes #17749

dcreager added a commit that referenced this pull request

Sep 10, 2025

@dcreager

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})