-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
void #3250
Comments
I don't think we should be configuring this at the level of an HTTP response, the response is an IO object so its behavior should be dictated by the consuming user by calling either |
Does calling |
read1
parameter to further methods
Brought this up in Discord too, copying here. I think |
It's not necessarily flawed by design, but it's yet another issue. We could either forbid |
The fact that |
The current version of urllib3's
Yes, "not having".
It is unclear what you see as the expected result of #3257 at the moment. |
@abebeos Thank you for your interest in contributing to urllib3. I wanted to provide some context into our project and feedback on your approach. I hope that my comment here won't discourage you and instead will help you be more effective towards your goals. Bounties are only assigned to tasks which have consensus from maintainers and are clearly defined solutions. Tasks which don't have consensus or aren't clearly defined (like this one) won't have bounties assigned until they are clear and agreed upon. It's okay to ask for an issue to be assigned a bounty once it meets this threshold. Speaking for myself, your approach of opening many issues and PRs all mentioning bounties does not jive well with me, and that may not have been your intent but it is the effect it's had on me. If your goal is to be paid a bounty (something that we want to do, we want to pay you!) you should be working on gaining consensus from maintainers by feeding our context or making concrete and concise proposals. Making tons of comments, linking to other places, marking things as outdated, proposing PRs and then closing them later, all of that gets in the way of maintainers understanding the problem and your proposed solution. We only have so many minutes in a day to spend on open source, so clarity and conciseness count. I also wanted to note from some of your PRs there is a sense of "merge if it's not incorrect, we can figure it out later". We don't take that approach with this project, doing something for no reason or an unclear reason is a negative outcome even if on the surface the PR is a no-op in terms of value to the project. We value intentionality over getting things done quickly, because one day many years in the future someone (unlikely to be you or me!) will have to know why we made certain changes (and we don't know which ones will matter!) Again, I hope you understand that this feedback comes from a place of wanting to see you succeed and to ultimately pay you to contribute to our project. Thanks again! |
No description provided.
The text was updated successfully, but these errors were encountered: