uranusjr commented on issue #41641: URL: https://github.com/apache/airflow/issues/41641#issuecomment-2487607130
I also played around the fixer implementation a bit: https://github.com/uranusjr/airflow2to3 Still a lot of room for improvement, but I think it is a good starting point. FWIW I also tried to create a Flake8 plugin for comparison, but got stuck resolving imports (we need to check whether a thing is actually from Airflow first to decide whether to emit errors for it). Flake8 does not seem to provide this out of the box (Ruff and LibCST each has an easy solution). I’m pretty sure there must be a solution, but the effort looking into this is probably not worthwhile unless we feel Ruff is too big a hurdle. -- 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]
