potiuk commented on PR #58497: URL: https://github.com/apache/airflow/pull/58497#issuecomment-3570466291
> Yes, that makes sense. Even after writing this first comment I realized it's not RM responsibility. I believe the title was a bit confusing there and suggested only typing is changed + as you said, the tests were not comprehensive enough. Sorry for the confusion. Also thanks for reminding about list of commits being in the docs, I usually used git diff in IDE, but maybe this will be easier. You can still use git diff - I do all the time - :) - it's really for the advantage of the users who might want to dig in but also those who are testing the provider. Also - this list of commits includes **ALL** committs even those where providers were moved in the repository - which is not at all given with `git diff` - git can easily track single file moves/renames, but good luck if you want to track directory moves - the list of commits we generate has the logic to also show changes for such moved providers -- 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]
