potiuk commented on PR #67052: URL: https://github.com/apache/airflow/pull/67052#issuecomment-5462505026
> @shahar1 I still cannot run example_cloud_composer on real Composer from here. > Your merge condition is that live run. I can leave this PR open if you or the Google team still plan to run it on the community project. I can also close it until someone actually has a run, and reopen or open a new PR then. > Which do you prefer? I don't want this sitting in the review queue if the test is not going to happen soon. I don't want to increase the review backlog. Then we should close it @Vamsi-klu -> you seem to open a lot of PRs with Claude Code but it seems you do not test it. If you cannot run system test now, it means your changes are theorethical and basically you are throwing testing over the wall to someone else. Which is not a good idea. I am closing all Google provider ones from you in this case - feel free to reopen your PRs for providers with a proof that you actuallly run the tests manually with real Google account. Also whenever we release a provider with such changes **you** are requested to installl an test such modified provider with **real** google service. The providers **are** supposed to be run with raal services they are accessing and not "theorethically" fixed. -- 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]
