potiuk commented on PR #46575: URL: https://github.com/apache/airflow/pull/46575#issuecomment-2645866227
> @potiuk You might want to take a look at this one Oh absolutely .. I knew we will have to review and update those (and possibly fix some things that we missed) - it was just sometimes a bit too **complex** to get it all right with new and old provider structures. I'd say what @jscheffl wrote is good idea - let's fix things and keep on rebasing those for the next few days (I hope) and merge right after we merge amazon and microsoft.azure ones - then I have the "cleanup" step to do where I will be removing a lot of the code tha handles both new and old providers, and I can make sure to also review the pre-commits to make sure they are "correct". i saw at least few places where pre-commits are "somewhat" testing things now - possibly skipping some of the stuff we wanted to catch so some fixes will be inevitable. -- 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]
