-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Automated Generate from openapi release-1.34 #4424
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Kubernetes Publisher <k8s-publishing-bot@users.noreply.github.com>
Signed-off-by: Kubernetes Publisher <k8s-publishing-bot@users.noreply.github.com>
Signed-off-by: Kubernetes Publisher <k8s-publishing-bot@users.noreply.github.com>
Signed-off-by: Kubernetes Publisher <k8s-publishing-bot@users.noreply.github.com>
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: yue9944882 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
So I was hoping to pick up the recent changes I made to add tlsServerName support to the code generator, but I think I need to up the commit version that we use in the generator, this did pick up some fluent updates, so I think we may as well merge it, but we probably also need to re-squash it or something so tha the CLA bot is happy. |
|
just to confirm, does this PR include the tls change you are looking for? note our generation workflow is based on the latest content in this file:
|
|
No, we need to wait for an OpenAPI generator release which includes it and then rev the version forward. We can still merge this though if we want looks like there are some set related fixes. |
No description provided.