Skip to content

Keep clients in sync - #3042

Merged
karpetrosyan merged 3 commits into
encode:masterfrom
karpetrosyan:keep-clients-in-sync
Jan 8, 2024
Merged

karpetrosyan merged 3 commits into
encode:masterfrom
karpetrosyan:keep-clients-in-sync

Conversation

@karpetrosyan

@karpetrosyan karpetrosyan commented Jan 5, 2024 •

Copy link
Copy Markdown
Contributor

Except for the async/await syntax, AsyncClient and Client are supposed to be the same.

I isolated them into separate files and used git diff --no-index --word-diff asyncclient.py syncclient.py to find and fix differences.

Comment thread httpx/_client.py Resolved
Comment thread httpx/_client.py Resolved
@lovelydinosaur

Copy link
Copy Markdown
Member

neat!

@lovelydinosaur lovelydinosaur added the refactor Issues and PRs related to code refactoring label Jan 8, 2024
@lovelydinosaur

Copy link
Copy Markdown
Member

(ready to merge. reminder to team, we're okay without a CHANGELOG.md entry on refactorings that don't represent a change in user-facing behavior.)

@karpetrosyan
karpetrosyan merged commit ca51b45 into encode:master Jan 8, 2024
This was referenced Feb 26, 2024
samclearman pushed a commit to titanmsp/httpx that referenced this pull request Apr 26, 2025
* Keep clients in sync

* Update httpx/_client.py

* Update httpx/_client.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Issues and PRs related to code refactoring

Development

Successfully merging this pull request may close these issues.

2 participants