Skip to content

Add docs on request instances to Requests compatibility guide - #823

Merged
florimondmanca merged 3 commits into
encode:masterfrom
srikanthccv:master
Mar 25, 2020
Merged

florimondmanca merged 3 commits into
encode:masterfrom
srikanthccv:master

Conversation

@srikanthccv

@srikanthccv srikanthccv commented Feb 24, 2020 •

Copy link
Copy Markdown
Contributor
  • Add prepare_request equivalent of httpx to compatibility document

  • Add difference b/w httpx.Request and requests.Request arguments to compatibility document

Fixes #795, fixes #794

* Add prepare_request equivalent of httpx to compatibility document

* Add difference b/w httpx.Request and requests.Request arguments to compatibility document
@srikanthccv

Copy link
Copy Markdown
Contributor Author

Hello @florimondmanca ,

I have tried to address the issues #794 and #795 with adding the documentation in this PR.

@florimondmanca florimondmanca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a ton! I left a few copy suggestions, but this is a nice step forward.

Comment thread docs/compatibility.md Outdated
Comment thread docs/compatibility.md Outdated
Comment thread docs/compatibility.md Outdated
* Remove Request arguments section
* Add new section Request instantiation with necessary info
Comment thread docs/compatibility.md Outdated
@florimondmanca florimondmanca added the docs Changes to the documentation label Feb 29, 2020
@florimondmanca florimondmanca changed the title Update docs/compatibility.md Add request instances to Requests compatibility guide Feb 29, 2020
@florimondmanca florimondmanca changed the title Add request instances to Requests compatibility guide Add docs on request instances to Requests compatibility guide Feb 29, 2020
@florimondmanca

Copy link
Copy Markdown
Contributor

Hi @lonewolf3739, since #836 is now merged seems like we should be able to update this PR and get it over the line?

Commit the suggested changes

Co-Authored-By: Florimond Manca <florimond.manca@gmail.com>
@srikanthccv

Copy link
Copy Markdown
Contributor Author

Hi, @florimondmanca I have updated the docs.

@florimondmanca florimondmanca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@florimondmanca
florimondmanca merged commit 5cf1acc into encode:master Mar 25, 2020
Kludex referenced this pull request in pydantic/httpx2 May 11, 2026
…tionNotAvailable (#823)

* add failing tests

* fix pooltimeout

* Revert "add failing tests"

This reverts commit cacc248c1fc7f968c5e40067fba615df1c056e2d.

* mark `if timeout < 0` as not covered, since the test is reverted

* Update CHANGELOG.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Changes to the documentation

Development

Successfully merging this pull request may close these issues.

httpx.Client misses the prepare_request method httpx.Request don't take the same arguments

2 participants