potiuk commented on PR #45020: URL: https://github.com/apache/airflow/pull/45020#issuecomment-2563020013
> > > I also just noticed it was on the "wrong" branch - but only because I had a test on v2-10-test running that was cancelled due to the merge. > > > > > > Apologies for that :) @potiuk fair enough. Is there any stance about removing the providers from `v2-10-test` to avoid such confusion? I assume that it hasn't been done until now for a reason, not sure what though considering that we continue maintaining them in `main`. > > No we just keep them here. Would be "too much" cleanup and CI has still a lot of dependencies and cross-checks. Actually it could be possible eventually. We have been removing all the "remnants" of provider tests from CI and I think with some future refactoring of providers into separate projects (working on it) - it could be possible eventually. I might attempt it as part of https://github.com/apache/airflow/issues/42632 and after https://github.com/apache/airflow/issues/44511 is completed. For now the remaining parts are: * "airflow" contains some tests that are expecting providers (executors and few others) as mentioned by @jscheffl * airflow "editable" build expects the "extras" to use provider's dependencies not providers itself (but that could be likely mitigated easily - and we could use the "released" providers in `v2-10-test` It's actually quite likely that for Airflow 3 we will have "non-providers" code only in `v3-0-test`. -- 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]
