Skip to content

Conversation

@jscheffl
Copy link
Contributor

@jscheffl jscheffl commented Nov 1, 2025

While implementing #57294 I realized that a couple of more ruff PLW rules might be interesting, this PR enables PLW1641

See also https://docs.astral.sh/ruff/rules/#warning-plw

Copy link
Contributor

@bugraoz93 bugraoz93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Shouldn't we include the rule (PLW1641) in pyproject.toml? Maybe it is coming as default that I am not 100% sure, wanted to point out .
Sorry, it is there, my bad! Was checking this through phone, something tricked me there 🤦

@jscheffl jscheffl merged commit 58fc860 into apache:main Nov 1, 2025
221 checks passed
@github-actions
Copy link

github-actions bot commented Nov 1, 2025

Backport failed to create: v3-1-test. View the failure log Run details

Status Branch Result
v3-1-test Commit Link

You can attempt to backport this manually by running:

cherry_picker 58fc860 v3-1-test

This should apply the commit to the v3-1-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

jscheffl added a commit to jscheffl/airflow that referenced this pull request Nov 1, 2025
(cherry picked from commit 58fc860)

Co-authored-by: Jens Scheffler <[email protected]>
jscheffl added a commit to jscheffl/airflow that referenced this pull request Nov 2, 2025
(cherry picked from commit 58fc860)

Co-authored-by: Jens Scheffler <[email protected]>
jscheffl added a commit to jscheffl/airflow that referenced this pull request Nov 2, 2025
(cherry picked from commit 58fc860)

Co-authored-by: Jens Scheffler <[email protected]>
jscheffl added a commit that referenced this pull request Nov 3, 2025
ephraimbuddy pushed a commit that referenced this pull request Nov 3, 2025
ephraimbuddy pushed a commit that referenced this pull request Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants