potiuk commented on pull request #20466: URL: https://github.com/apache/airflow/pull/20466#issuecomment-999887001
> It's not the line of code that worries me but the no replies to the issues & PRs. But yeah not a big deal we can remove it if when it breaks or one of us can fork it or help maintain it Yeah. I saw it too. And good you checked. Maybe not "responsive" enough but I think most of those issues are for "pre-python 3" and some very specific requests. And we can always remove it if we see problems - luckily it's just "when we make commit" type of dependency - we don't even add it to airflow's setup.* files :). The main reason for me to add it is that I plan to add "execute(context: "Context") all over the providers automatically and that results in some manual 'remove imports" - but I figured it could also helped in the past where I had to manually remove some imports but pre-commits could do it for me :) Let's try and see - and we can always remove it if we find it causing problems. -- 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]
