Closed ityuhui closed 3 years ago
/cc @brendandburns
This implementation looks fine to me. What are the open questions?
Hi @brendandburns
No more question except the code review. I will create the PR for apiClient.{h,c} in openapi-generator as you have agreed to the code change.
This looks good to me.
/lgtm /approve
Updated the code after https://github.com/kubernetes-client/c/pull/37 is merged.
Hi @brendandburns Could you please approve the new code change to trigger k8s-ci-robot to merge this PR ?
/lgtm /approve
We may want to add a section documenting how to do multi-threaded programming with the client.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: brendandburns, ityuhui
The full list of commands accepted by this bot can be found here.
The pull request process is described here
We may want to add a section documenting how to do multi-threaded programming with the client.
Yes. I plan to address it:
The draft to fix #34
We can talk about the implementation here. And some code changes will commit to openapi-generator project and merge back.