Skip to content

[3.8] bpo-10544: Disallow "yield" in comprehensions and generator expressions. #4564

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

Conversation

serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Nov 25, 2017

@serhiy-storchaka serhiy-storchaka added the type-feature A feature request or enhancement label Nov 25, 2017
@serhiy-storchaka serhiy-storchaka changed the title bpo-10544: Disallow "yield" in comprehensions and generator expressions. [3.8] bpo-10544: Disallow "yield" in comprehensions and generator expressions. Nov 27, 2017
@serhiy-storchaka serhiy-storchaka force-pushed the disallow-yield-in-comprehencions branch from f4f684d to 6d5e555 Compare December 1, 2017 07:04
Copy link
Contributor

@ncoghlan ncoghlan left a comment

Choose a reason for hiding this comment

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

Just adding a blocking review to ensure we don't merge this until after the 3.7 maintenance branch has been created.

@bedevere-bot
Copy link

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@serhiy-storchaka
Copy link
Member Author

Could you please look on this again Nick?

Copy link
Contributor

@ncoghlan ncoghlan left a comment

Choose a reason for hiding this comment

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

LGTM, and the main branch is now open for 3.8 commits.

@@ -0,0 +1,2 @@
Yield expressions now are disallowed in comprehensions and generator
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor wording tweak: now are -> are now

@serhiy-storchaka
Copy link
Member Author

Maybe you can suggest better wording for What's New?

@ncoghlan
Copy link
Contributor

ncoghlan commented Feb 4, 2018

The What's New wording seemed fine to me.

@serhiy-storchaka serhiy-storchaka merged commit 07ca9af into python:master Feb 4, 2018
@serhiy-storchaka serhiy-storchaka deleted the disallow-yield-in-comprehencions branch February 4, 2018 08:53
@serhiy-storchaka
Copy link
Member Author

Thank you @ncoghlan for your review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-feature A feature request or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants