[proxy] web.archive.org← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public

Conversation

Copy link

remilapeyre commented Jan 11, 2019

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
Copy link

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.

Copy link
Author

I have made the requested changes; please review again

Copy link

Thanks for making the requested changes!

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

Copy link
Member

ethanfurman left a comment

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. :)

Copy link

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.

Copy link
Author

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

Copy link

Thanks for making the requested changes!

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

Copy link
Member

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

Copy link
Member

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

Copy link
Member

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.

Copy link
Contributor

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
Copy link

GH-11669 is a backport of this pull request to the 3.7 branch.

vstinner pushed a commit that referenced this pull request Jan 30, 2019
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

Comments