jscheffl commented on PR #36517: URL: https://github.com/apache/airflow/pull/36517#issuecomment-1873268073
> Once concern is backwards compatibility (and I think that's where @kaxil's solution came from). > > But yeah, I think you are right @jscheffl @dirrao - probably return value is not **teally** compatibility promise (and we can treat it as bugfix). We've already done the same in #35378 (and It was me who proposed skipping the flag and treating it as bugfix :D ... Now I remember . So if we had it two times already would be even a good "quality excercise" to double check that all CLI commands exit with a proper error code in case of error :-D -- 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]
