Skip to content
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

Refactor API client #1018

Merged
merged 8 commits into from
Jun 8, 2022
Merged

Refactor API client #1018

merged 8 commits into from
Jun 8, 2022

Conversation

therve
Copy link
Contributor

@therve therve commented May 25, 2022

This moves thread support from the main client, and extract the behavior
flags in the configuration.

@therve therve requested a review from a team as a code owner May 25, 2022 16:26
@github-actions github-actions bot added the documentation Improvements or additions to documentation label May 25, 2022
@therve therve force-pushed the therve/v2-branch branch from eb99e3f to e704aae Compare May 30, 2022 07:08
@therve therve force-pushed the therve/v2-branch branch from 487b382 to 82f840a Compare June 3, 2022 07:58
therve and others added 5 commits June 8, 2022 13:31
This moves thread support from the main client, and extract the behavior
flags in the configuration.
Co-authored-by: NouemanKHAL <noueman.khalikine@datadoghq.com>
@therve therve force-pushed the therve/v2-branch branch from 908369c to 1f3661d Compare June 8, 2022 11:33
@therve therve merged commit 76802a2 into master Jun 8, 2022
@therve therve deleted the therve/v2-branch branch June 8, 2022 12:48
github-actions bot pushed a commit that referenced this pull request Jun 8, 2022
* Refactor API client

This moves thread support from the main client, and extract the behavior
flags in the configuration.

* Small tweak

* Implement close

* Apply suggestions from code review

Co-authored-by: NouemanKHAL <noueman.khalikine@datadoghq.com>

* Rebase

* Doc tweak

* Fix one docstring escape

* Make zstd optional

Co-authored-by: NouemanKHAL <noueman.khalikine@datadoghq.com> 76802a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants