Skip to content

fix: remove reliance on SyncClient and use Client directly from httpx #607

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

Merged
merged 1 commit into from
Jun 17, 2025

Conversation

silentworks
Copy link
Contributor

What kind of change does this PR introduce?

Bug fixes

  • fix: add deprecation warnings for SyncClient
  • test: update to use Client instead of SyncClient
  • chore: remove deprecated classes Client and GetRequestBuilder

What is the current behavior?

Please link any relevant issues here.

What is the new behavior?

Feel free to include screenshots if it includes visual changes.

Additional context

Add any other context or screenshots.

fix: add deprecation warnings for SyncClient
test: update to use Client instead of SyncClient
chore: remove deprecated classes Client and GetRequestBuilder
@silentworks silentworks requested a review from grdsdev as a code owner June 16, 2025 16:30
@coveralls
Copy link

Pull Request Test Coverage Report for Build 15686471164

Details

  • 43 of 43 (100.0%) changed or added relevant lines in 17 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.07%) to 96.217%

Totals Coverage Status
Change from base Build 15651689878: 0.07%
Covered Lines: 1882
Relevant Lines: 1956

💛 - Coveralls

@silentworks silentworks merged commit 021f1b6 into main Jun 17, 2025
11 checks passed
@silentworks silentworks deleted the silentworks/fix-remove-syncclient branch June 17, 2025 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants