jscheffl opened a new pull request, #57659:
URL: https://github.com/apache/airflow/pull/57659

   While implementing https://github.com/apache/airflow/pull/57294 I realized 
that a couple of more ruff PLW rules might be interesting, this PR enables 
PLW1509 - and adds fixes as needed.
   
   Fun stuff that we had especially in standard provider really a long time 
deprecated fucntion. I was able to trace the usage back to the year 2017 in 
https://github.com/apache/airflow/commit/ca961042c146d49504e00e4abefc7779f0747782
 where it was initially added which was closing 
https://issues.apache.org/jira/browse/AIRFLOW-1745 (for somebody still having 
access to this...).
   
   So for the change in standard provider - as far as I see the docs all 
supported python versions allow a replacement but I'd like really a "gray hair 
person" expertise review by e.g. @ashb @kaxil @potiuk @dstandish (including 
these who still have hair...) if the replacement code is really good enough.
   
   See also https://docs.astral.sh/ruff/rules/#warning-plw


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to