-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
PEP 655: Clarify there are no grammar changes or runtime enforcement. #2388
Conversation
Happy to merge this unless you want feedback from others first. |
@JelleZijlstra Let's leave this PR open for a day longer, since it's a high-diff change. I'll plan to poke the thread for merge tomorrow morning (Pacific time). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some grammar, punctuation and phrasing fixes
Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
Did finish applying many grammar, punctuation, and phrasing adjustments from @CAM-Gerlach . Thanks for taking a close look @CAM-Gerlach ! |
…n whose absense forces a reparse of the sentence upon first reading
This branch looks good to me with the above revisions and I think is ready to be merged. |
Thanks to Petr Viktorin (@encukou) for this feedback.
Also did apply many grammar, punctuation, and phrasing adjustments from @CAM-Gerlach .