-
-
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 639: edits from review #1628
Conversation
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.
@di Thank you ++ .. I made a few suggestions myself that you may want to merge in.
Co-authored-by: Philippe Ombredanne <pombredanne@gmail.com>
@pombredanne Done! Thanks! |
@python/pep-editors dear editors, it would be great if you could merge this PR at this stage. |
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.
One minor wording suggestion.
If you disagree, I'm also happy to merge without the suggested change.
Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
@JelleZijlstra Thanks! |
* Fix monospaces * The 'license expression' term shouldn't be monospaced/capitalized * Rephrasing * Fix capitalization of titles * Make it clear when we're talking about the Classifiers field vs classifiers * More monospace fixes * Edits for clarity * Fix 'License ::' prefix * Update link * More tweaks for clarity * Fix broken ref Co-authored-by: Philippe Ombredanne <pombredanne@gmail.com> Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
* Fix monospaces * The 'license expression' term shouldn't be monospaced/capitalized * Rephrasing * Fix capitalization of titles * Make it clear when we're talking about the Classifiers field vs classifiers * More monospace fixes * Edits for clarity * Fix 'License ::' prefix * Update link * More tweaks for clarity * Fix broken ref Co-authored-by: Philippe Ombredanne <pombredanne@gmail.com> Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
(cc @pombredanne)