Skip to content
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

Jump to bottom

Revert "move a dot outside double quotes" #20175

Merged
merged 2 commits into from May 21, 2020
Merged

Revert "move a dot outside double quotes" #20175

merged 2 commits into from May 21, 2020

Conversation

deronnax
Copy link
Sponsor 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
@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels May 18, 2020
@pganssle
Copy link
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 changes
Copy link
Member

@pganssle pganssle left a comment

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

@bedevere-bot
Copy link

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
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
Sponsor Contributor Author

deronnax commented May 21, 2020

here you go

zware
zware approved these changes May 21, 2020
View changes
@deronnax deronnax requested a review from pganssle May 21, 2020
@pganssle pganssle merged commit 2e76820 into python:master May 21, 2020
4 checks passed
@miss-islington
Copy link
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

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

bedevere-bot commented May 21, 2020

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

@pganssle
Copy link
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

@zware zware

@abalkin 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.

None yet
7 participants
@deronnax @pganssle @bedevere-bot @JulienPalard @miss-islington @zware @the-knights-who-say-ni