Skip to content

move a dot outside double quotes #20007

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

Merged

Conversation

deronnax
Copy link
Contributor

@deronnax deronnax commented May 8, 2020

in the previous version of the sentence, the dot was rightly outside the quotes.

@JulienPalard
Copy link
Member

LGTM, thanks @deronnax!

@JulienPalard JulienPalard merged commit 4eba677 into python:master May 17, 2020
@miss-islington
Copy link
Contributor

Thanks @deronnax for the PR, and @JulienPalard for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 17, 2020
(cherry picked from commit 4eba677)

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

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

miss-islington added a commit that referenced this pull request May 17, 2020
(cherry picked from commit 4eba677)

Co-authored-by: Mathieu Dupuy <mathieu.dupuy@doctolib.com>
@zware
Copy link
Member

zware commented May 18, 2020

Technically, I believe this change is grammatically incorrect even though it is something that I frequently use myself. It might be better to just avoid the situation with something like Date and time objects may be categorized as "aware" or "naive" depending on whether or not they include timezone information., or use bold or italic emphasis rather than double quotes.

@deronnax
Copy link
Contributor Author

it's depressing but it looks @zware is right : https://www.thepunctuationguide.com/quotation-marks.html.
A revert should be considered 😞

deronnax added a commit to deronnax/cpython that referenced this pull request May 18, 2020
pganssle pushed a commit that referenced this pull request May 21, 2020
This is more informative and avoids the question of whether the period should go inside or outside the quotation marks.

See also GH-20007.
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 21, 2020
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>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 21, 2020
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>
miss-islington added a commit that referenced this pull request May 22, 2020
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
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 pushed a commit to arturoescaip/cpython that referenced this pull request May 24, 2020
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
Labels
docs Documentation in the Doc dir skip issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants