potiuk commented on PR #42081: URL: https://github.com/apache/airflow/pull/42081#issuecomment-2442051787
> > > Just one thing about compat provider reuse. > > > > > > yeah am updating imports to use compat. > > I've added the imports to the compat provider, but I've noticed test failures occurring in selective checks. This is likely because all provider imports are now referenced directly from the compat provider. So, when we revert to standard provider direct imports, we may need to update all the selective check tests accordingly. > No - it's ok - you need to fix the tests accordingly. Those tests are testing whether the right set of "dependent" providers are selected when selective checks is running - so when you start depending on the standard provider and when standard provider depends on some other provider., the tests start to expect changes in the depednencies, so what needs to be done is to update the tests to reflect that. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
