Skip to content

bpo-46487: Add get_write_buffer_limits to Write and _SSLProtocol transports#30958

Merged
asvetlov merged 6 commits intopython:mainfrom
mooncell07:fix-issue-46487
Feb 1, 2022
Merged

bpo-46487: Add get_write_buffer_limits to Write and _SSLProtocol transports#30958
asvetlov merged 6 commits intopython:mainfrom
mooncell07:fix-issue-46487

Conversation

@mooncell07
Copy link
Contributor

@mooncell07 mooncell07 commented Jan 27, 2022

@miss-islington
Copy link
Contributor

Sorry, I can't merge this PR. Reason: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information..

@asvetlov
Copy link
Contributor

Should wait until @pablogsal resolve issue with failed buildbots

@asvetlov asvetlov self-assigned this Jan 29, 2022
@kumaraditya303
Copy link
Contributor

The buildbots are fixed on main branch now.

@mooncell07
Copy link
Contributor Author

Um, sorry did I do something wrong

@ambv
Copy link
Contributor

ambv commented Feb 1, 2022

Nope, all good, @miss-islington went into an infinite loop applying and removing the automerge label. It took me a second to fix it, hence the temporary DO-NOT-MERGE label.

@AlexWaygood
Copy link
Member

Um, sorry did I do something wrong

I managed to trigger an anxiety attack in the miss-islington bot, because I tried to reapply the "automerge" label, to get her to merge your PR. But I'm only a CPython triager, so I don't have merge privileges, hence the anxiety attack.

Nothing to do with you — just me and the bots having a totally normal morning :) thanks @ambv for the quick fix!!

@miss-islington
Copy link
Contributor

Thanks @mooncell07 for the PR, and @asvetlov for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

@asvetlov
Copy link
Contributor

asvetlov commented Feb 1, 2022

Thanks!

@bedevere-bot
Copy link

GH-31055 is a backport of this pull request to the 3.10 branch.

@bedevere-bot
Copy link

GH-31056 is a backport of this pull request to the 3.9 branch.

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.

8 participants