bpo-35717: Fix KeyError exception raised when using enums and compile by remilapeyre ยท Pull Request #11523 ยท python/cpython

@remilapeyre

@remilapeyre remilapeyre changed the title bpo-17467: Fix KeyError exception raised when using enums and compile bpo-35717: Fix KeyError exception raised when using enums and compile

Jan 11, 2019

vstinner

ethanfurman

@bedevere-bot

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@remilapeyre

I have made the requested changes; please review again

@bedevere-bot

Thanks for making the requested changes!

@ethanfurman: please review the changes made to this pull request.

ethanfurman

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Now add yourself to Misc/ACKS. :)

@bedevere-bot

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@remilapeyre

Thanks @ethanfurman, I have made the requested changes; please review again.

@bedevere-bot

Thanks for making the requested changes!

@ethanfurman: please review the changes made to this pull request.

@ethanfurman

Thank you. As soon as I get this on my machine for final checks I'll move it forward. :)

@vstinner

Now add yourself to Misc/ACKS. :)

Honestly, I don't think that it it's still important since we moved to Git: the author is know tracked in Git.

Note: There is a "thanks.python.org" project similar to https://thanks.rust-lang.org/ cc @pablogsal

@pablogsal

Note: There is a "thanks.python.org" project similar to https://thanks.rust-lang.org/ cc @pablogsal

Sadly, it was cancelled as in the core dev sprint it was decided not worth the maintenance cost.

ethanfurman

@miss-islington

Thanks @remilapeyre for the PR ๐ŸŒฎ๐ŸŽ‰.. I'm working now to backport this PR to: 3.7.
๐Ÿ๐Ÿ’โ›๐Ÿค–

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request

Jan 24, 2019

@miss-islington

@bedevere-bot

vstinner pushed a commit that referenced this pull request

Jan 30, 2019

@miss-islington @vstinner