-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
PEP 8: Allow both one and two spaces after period #3027
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
1bf21cd
to
e66eb53
Compare
FWIW you're missing a |
Right, I deliberately removed the |
Authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
e66eb53
to
eb4f82c
Compare
Can we make it self-referential? :-) |
Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
Re: #3024 (comment) Do we also want to add something to say it "doesn't require consistency (not even within a paragraph)"? Whilst editing this file, let's also remove the redundant Version and Last-Modified headers, and the Local Variables footer, as we've gradually been doing elsewhere. |
Has that been enough of a problem (for code comments at least, which is what this mention is about and not the reST document that triggered this) that it's worth an explicit sentence stating so? Consistency is nice, just not required or worth changes to existing text that isn't already rewritten. |
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.
LGTM, thanks @erlend-aasland and @hugovk !
* main: PEP 8: Allow both one and two spaces after period (python#3027) Meta: Further tweak PR template titles to get the right order (python#3001)
Authored-by: Hugo van Kemenade hugovk@users.noreply.github.com