Skip to content

move a dot outside double quotes#20007

Merged
JulienPalard merged 1 commit intopython:masterfrom
deronnax:library_datetime_move_dot_outside_quotes
May 17, 2020
Merged

move a dot outside double quotes#20007
JulienPalard merged 1 commit intopython:masterfrom
deronnax:library_datetime_move_dot_outside_quotes

Conversation

@deronnax
Copy link
Copy Markdown
Contributor

@deronnax deronnax commented May 8, 2020

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

@JulienPalard
Copy link
Copy Markdown
Member

LGTM, thanks @deronnax!

@JulienPalard JulienPalard merged commit 4eba677 into python:master May 17, 2020
@miss-islington
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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