Lee-W commented on PR #50745:
URL: https://github.com/apache/airflow/pull/50745#issuecomment-2889392701

   > > > Okay as fix, otherwise the if conditions in that old migration code 
would need to be optimized for readability :-D
   > > 
   > > 
   > > Just get back to my laptop before bed 🙌 . removed one layer of nested 
if-else. please check whether it's a bit better 👀
   > 
   > Better! Thannks for un-rolling the if - but CI is not convinced. So maybe 
tomorrow after some sleep you will make it happen :-D
   
   Diff ruff version 🤦‍♂️ I'm still using the one on main. I extracted that 
logic to the utils as it was used in `0100` as well.


-- 
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