potiuk commented on PR #69749: URL: https://github.com/apache/airflow/pull/69749#issuecomment-5462546846
There are quite a few issues in here - one is that you introduce whole provider breaking change with one simple thing, two is that you should not add the version in changelog yourself, third - you should make sure to run system tests with Google changes - which you claim you can't - so this change is **again** theorethical.. Feel free to reopen it with system tests and logs of those being executed if you want to follow it. https://github.com/apache/airflow/pull/67052#issuecomment-5462505026 I have nothing against using AI to generate fixes, though your pattern of contributions seems that you have no understanding or stake in any of the multiple projects you spend your tokens on. If you do **not** run Airflow, and do **not** test Providers with actual services, any substrantial change like that is just thorethical. Please refrain from doing it - there are other places in Airflow that can be easily tested locally.. -- 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]
