-
-
Notifications
You must be signed in to change notification settings - Fork 857
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
Rename PoolLimits to Limits #1113
Conversation
@@ -418,8 +419,7 @@ class Client(BaseClient): | |||
URLs. | |||
* **timeout** - *(optional)* The timeout configuration to use when sending | |||
requests. | |||
* **pool_limits** - *(optional)* The connection pool configuration to use | |||
when determining the maximum number of concurrently open HTTP connections. | |||
* **limits** - *(optional)* The limits configuration to use. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haven't found a better description of the parameter that's not redundant with the info in the Limits
docstring. I don't think it's included in the Developer Interface yet so that could be a nice follow-up, and then cross-link it here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we could say something like... "Allows for configuration options, such as the maximum number of open connections to allow." Just to give a bit of an indication what we mean by "limits" here(?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's pull this in for now as it is. Welcome to follow up with docstring improvements later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, yup!
Fixes #1094
Change the following with gentle deprecation:
httpx.PoolLimits(...)
becomeshttpx.Limits(...)
Client(pool_limits=...)
becomesClient(limits=...)
AsyncClient(pool_limits=...)
becomesAsyncClient(limits=...)