pierrejeambrun commented on code in PR #34805: URL: https://github.com/apache/airflow/pull/34805#discussion_r1352817998
########## clients/gen/common.sh: ########## @@ -17,7 +17,7 @@ # specific language governing permissions and limitations # under the License. -OPENAPI_GENERATOR_CLI_VER=5.4.0 +OPENAPI_GENERATOR_CLI_VER=7.0.1 Review Comment: I am afraid @ephraimbuddy is right. On the other hand this starts to be a little blocking. For instance for the go client, and other client issues where we need to manually fix things. (Missing supports for certain keywords in the spec etc...). We didn't discuss this but maybe we could do a major release for clients, independantly of airflow major release, just an idea, to not have clients stuck for a long long time. Like update the client versionning policy to allow that. I don't think major version needs to match between client and airflow core. (It will be a little harder to handle but worth I believe). If this sounds like an idea worth discussing, I can bring that to the dev list. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
