-
Notifications
You must be signed in to change notification settings - Fork 46
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
Support multi-thread #36
Support multi-thread #36
Conversation
/cc @brendandburns |
cf33e7e
to
99c0ed1
Compare
This implementation looks fine to me. What are the open questions? |
No more question except the code review. |
This looks good to me. |
/lgtm |
99c0ed1
to
493a027
Compare
Updated the code after #37 is merged. Hi @brendandburns |
/lgtm 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
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
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.