Skip to content

Conversation

@roy-pstr
Copy link
Contributor

@roy-pstr roy-pstr commented Nov 28, 2022

The starting point for contributions should usually be a discussion

Simple documentation typos may be raised as stand-alone pull requests, but otherwise please ensure you've discussed your proposal prior to issuing a pull request.

This will help us direct work appropriately, and ensure that any suggested changes have been okayed by the maintainers.

@Kludex Kludex added this to the Version 0.23.0 milestone Dec 2, 2022
@Kludex
Copy link
Owner

Kludex commented Dec 3, 2022

Thanks @roy-pstr ! :)

@Kludex
Copy link
Owner

Kludex commented Dec 3, 2022

Just for record keeping...

Even though it's possible to add the headers to the TestClient after the client is instantiated, it's more comfortable to add the headers' parameter. It's not only about practicality, but I believe people will make a good use of this feature.

Also, do not take this PR as an argument to add the other parameters supported by httpx client.

@Kludex Kludex merged commit b77a41d into Kludex:master Dec 3, 2022
aminalaee pushed a commit that referenced this pull request Feb 13, 2023
Co-authored-by: Marcelo Trylesinski <[email protected]>
Co-authored-by: Roy Pasternak <[email protected]>
Co-authored-by: Roy Pasternak <roy-pstr>
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.

4 participants