potiuk commented on PR #30424: URL: https://github.com/apache/airflow/pull/30424#issuecomment-1493149189
I also correctedalm the Ci references. breeze is deliberately in Airflwo repo so that they can evolve together and so we do not have to care about backwards compatibility. And this paran is norlstly use d in CI - most people even if run parallel tests they would run it with default value (all tests). We have no expectations nor need about breeze being backwards compatible :) I understand where you're coming from but i think it is really the matter of wher you get 'common' part we are reusing a lot of code under the hood, and it's a bit arbitrary decision how to name parameters that internally might even consist of similar 'blocks' In this case i'd say separating it MORE is better because it removes confusion and typos -- 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]
