Skip to content

Revert "move a dot outside double quotes"#20175

Merged
pganssle merged 2 commits intopython:masterfrom
deronnax:revert-20007-library_datetime_move_dot_outside_quotes
May 21, 2020
Merged

Revert "move a dot outside double quotes"#20175
pganssle merged 2 commits intopython:masterfrom
deronnax:revert-20007-library_datetime_move_dot_outside_quotes

Conversation

@deronnax
Copy link
Copy Markdown
Contributor

@deronnax deronnax commented May 18, 2020

As seen in #20007, contrary to what one would expect, in American English, dots go into quotes.
see https://www.thepunctuationguide.com/quotation-marks.html.

Reverts #20007

@deronnax deronnax requested review from abalkin and pganssle as code owners May 18, 2020 12:17
@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels May 18, 2020
@pganssle
Copy link
Copy Markdown
Member

pganssle commented May 18, 2020

I think it's basically controversial how to do this, I like @zware's suggestion to reword the question so as to obviate the need to take a stance on this, I think:

Date and time objects may be categorized as "aware" or "naive" depending on whether or not they include timezone information.

both solves the problem and improves the reader's comprehension, so I call it a win-win.

pganssle
pganssle requested changes May 18, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@pganssle pganssle left a comment

Choose a reason for hiding this comment

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

Rather than revert, let's adopt @zware's suggestion.

@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented May 18, 2020

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.

@JulienPalard
Copy link
Copy Markdown
Member

JulienPalard commented May 18, 2020

I concour with @pganssle, using a workaround will save everybody the time to understand why the dot is here (allowing the brain to focus on what is said by the doc, which is good).

@deronnax
Copy link
Copy Markdown
Contributor Author

deronnax commented May 21, 2020

here you go

@deronnax deronnax requested a review from pganssle May 21, 2020 20:19
@pganssle pganssle merged commit 2e76820 into python:master May 21, 2020
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented May 21, 2020

Thanks @deronnax for the PR, and @pganssle for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8, 3.9.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 21, 2020
@miss-islington
Reword aware/naive introduction sentence (pythonGH-20175)
8885bd4 
This is more informative and avoids the question of whether the period should go inside or outside the quotation marks.

See also pythonGH-20007.
(cherry picked from commit 2e76820)

Co-authored-by: Mathieu Dupuy <mathieu.dupuy@doctolib.com>
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented May 21, 2020

GH-20298 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 21, 2020
@miss-islington
Reword aware/naive introduction sentence (pythonGH-20175)
ba75fdf 
This is more informative and avoids the question of whether the period should go inside or outside the quotation marks.

See also pythonGH-20007.
(cherry picked from commit 2e76820)

Co-authored-by: Mathieu Dupuy <mathieu.dupuy@doctolib.com>
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented May 21, 2020

GH-20299 is a backport of this pull request to the 3.8 branch.

@pganssle
Copy link
Copy Markdown
Member

pganssle commented May 21, 2020

Thanks @deronnax et al!

miss-islington added a commit that referenced this pull request May 22, 2020
@miss-islington
Reword aware/naive introduction sentence (GH-20175)
0714c48 
This is more informative and avoids the question of whether the period should go inside or outside the quotation marks.

See also GH-20007.
(cherry picked from commit 2e76820)

Co-authored-by: Mathieu Dupuy <mathieu.dupuy@doctolib.com>
miss-islington added a commit that referenced this pull request May 22, 2020
@miss-islington
Reword aware/naive introduction sentence (GH-20175)
c1d7768 
This is more informative and avoids the question of whether the period should go inside or outside the quotation marks.

See also GH-20007.
(cherry picked from commit 2e76820)

Co-authored-by: Mathieu Dupuy <mathieu.dupuy@doctolib.com>
arturoescaip pushed a commit to arturoescaip/cpython that referenced this pull request May 24, 2020
@arturoescaip
Reword aware/naive introduction sentence (pythonGH-20175)
1c83402 
This is more informative and avoids the question of whether the period should go inside or outside the quotation marks.

See also pythonGH-20007.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pganssle pganssle pganssle approved these changes

@zware zware zware approved these changes

@abalkin abalkin Awaiting requested review from abalkin

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@deronnax @pganssle @bedevere-bot @JulienPalard @miss-islington @zware @the-knights-who-say-ni