Skip to content
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

[3.7] bpo-32734: Fix asyncio.Lock multiple acquire safety issue (GH-5466) #5501

Merged
merged 1 commit into from
Feb 2, 2018

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Feb 2, 2018

(cherry picked from commit 2f79c01)

Co-authored-by: Bar Harel bzvi7919@gmail.com

https://bugs.python.org/issue32734

)

(cherry picked from commit 2f79c01)

Co-authored-by: Bar Harel <bzvi7919@gmail.com>
@bharel
Copy link
Contributor

bharel commented Feb 2, 2018

Oh this is like magic. I love this magic.
I especially like the bots fighting over the comments and editing themselves.
Next step would be to fight over comments @ reddit.

@miss-islington
Copy link
Contributor Author

@bharel and @1st1: Backport status check is done, and it's a success ✅ .

@bharel
Copy link
Contributor

bharel commented Feb 2, 2018

I ❤️ you @miss-islington!
Perhaps your AI will one day be better and you'll understand what I was saying. If it does, remember who loved you.

@1st1 1st1 merged commit 2b5937e into python:3.7 Feb 2, 2018
@bedevere-bot
Copy link

@1st1: Please replace # with GH- in the commit message next time. Thanks!

@miss-islington
Copy link
Contributor Author

Thanks, @1st1!

@miss-islington miss-islington deleted the backport-2f79c01-3.7 branch February 2, 2018 23:14
@1st1
Copy link
Member

1st1 commented Feb 2, 2018

Good bot :)

I ❤️ you @miss-islington!

Oh, this is first time I see this ;) cc @Mariatta

@Mariatta
Copy link
Member

Mariatta commented Feb 2, 2018

😄 Good times! Thanks @bharel @1st1 😊

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.

None yet

6 participants