Skip to content

Make 'request' non-optional on responses - #666

Merged
florimondmanca merged 3 commits into
masterfrom
non-optional-request-for-response
Dec 21, 2019
Merged

Make 'request' non-optional on responses#666
florimondmanca merged 3 commits into
masterfrom
non-optional-request-for-response

Conversation

@florimondmanca

Copy link
Copy Markdown
Contributor

Prompted by https://github.com/encode/httpx/pull/665/files#r360601429: response.request is only ever None in our tests, so let's make it non-optional to provide better type-checking UX on the user side.

@florimondmancaflorimondmanca added the user-experience Ensuring that users have a good experience using the library label Dec 20, 2019
@florimondmanca
florimondmanca requested a review from a teamDecember 20, 2019 23:11
@florimondmancaflorimondmanca changed the title Non optional request for responseMake 'request' non-optional on responsesDec 20, 2019

@lovelydinosaurlovelydinosaur 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.

Probably not a bad idea, yup!

@lovelydinosaurlovelydinosaur mentioned this pull request Dec 21, 2019
@florimondmanca
florimondmanca merged commit 56c8eda into masterDec 21, 2019
@florimondmanca
florimondmanca deleted the non-optional-request-for-response branch December 21, 2019 14:38
@florimondmancaflorimondmanca mentioned this pull request Dec 29, 2019
Kludex referenced this pull request in pydantic/httpx2 May 11, 2026
Bumps [mkdocs-material](https://github.com/squidfunk/mkdocs-material) from 8.5.7 to 9.1.5.
- [Release notes](https://github.com/squidfunk/mkdocs-material/releases)
- [Changelog](https://github.com/squidfunk/mkdocs-material/blob/master/CHANGELOG)
- [Upgrade guide](https://github.com/squidfunk/mkdocs-material/blob/master/docs/upgrade.md)
- [Commits](squidfunk/mkdocs-material@8.5.7...9.1.5)
---
updated-dependencies:
- dependency-name: mkdocs-material
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

user-experienceEnsuring that users have a good experience using the library

Development

Successfully merging this pull request may close these issues.

2 participants

@florimondmanca@lovelydinosaur