potiuk commented on PR #33602: URL: https://github.com/apache/airflow/pull/33602#issuecomment-1697133068
> I'm still don't get why `Breeze unit tests` failed, I've check it locally on that branch and it also complains. Maybe test doesn't like that changes happen in two provider in one go, or complain about new submodule `protocols` into the amazon provider `docker` is missing in the list of provider that shoudl be afffected when "amazon" changes: It does not expect docker because previously `docker` package was not dependent on amazon. It should also fail locally for you as long as you run pre-commit and generate the "generated/provider_dependencies.json" - this is where breeze selective checks finds out which provider tests should be run if another provider (that it depends on or dependency is the other way) changes. This is the way to determine minium set of provider tests that should be run for the PR (and get maximum chance of not getting broken main). -- 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]
