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

Erase some unnecessary quotes on data model doc #113521

Merged
merged 2 commits into from
Feb 25, 2024

Conversation

adorilson
Copy link
Contributor

@adorilson adorilson commented Dec 27, 2023

This is a PR before I start to work on #111746


📚 Documentation preview 📚: https://cpython-previews--113521.org.readthedocs.build/

@bedevere-app bedevere-app bot added docs Documentation in the Doc dir skip news awaiting review labels Dec 27, 2023
adorilson and others added 2 commits December 28, 2023 00:10
The check about the f argument type was removed in this commit:
python@2c94aa5

Thanks for Pedro Arthur Duarte (pedroarthur.jedi at gmail.com) by the help with
this bug.
@@ -266,4 +266,3 @@ Examples::

`NumPy <https://numpy.org/>`_
The NumPy package defines another array type.

Copy link
Member

Choose a reason for hiding this comment

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

Removing the extra newline is correct, but extraneous to the issue. I am leaving it, but normally better not to toss such things in.

@@ -34,7 +34,7 @@ represented by objects.)

Every object has an identity, a type and a value. An object's *identity* never
changes once it has been created; you may think of it as the object's address in
memory. The ':keyword:`is`' operator compares the identity of two objects; the
memory. The :keyword:`is` operator compares the identity of two objects; the
Copy link
Member

Choose a reason for hiding this comment

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

I suspect the quotes predate the :keyword: role tag.

@terryjreedy terryjreedy added needs backport to 3.11 only security fixes needs backport to 3.12 bug and security fixes labels Feb 25, 2024
@terryjreedy terryjreedy merged commit f745586 into python:main Feb 25, 2024
26 checks passed
@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 25, 2024
Thanks to Pedro Arthur Duarte (pedroarthur.jedi at gmail.com) for help with this bug.
(cherry picked from commit f745586)

Co-authored-by: Adorilson Bezerra <adorilson@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Feb 25, 2024

GH-115896 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Feb 25, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 25, 2024
Thanks to Pedro Arthur Duarte (pedroarthur.jedi at gmail.com) for help with this bug.
(cherry picked from commit f745586)

Co-authored-by: Adorilson Bezerra <adorilson@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Feb 25, 2024

GH-115897 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Feb 25, 2024
terryjreedy pushed a commit that referenced this pull request Feb 25, 2024
…115896)

Thanks to Pedro Arthur Duarte (pedroarthur.jedi at gmail.com) for help with this bug.

(cherry picked from commit f745586)

Co-authored-by: Adorilson Bezerra <adorilson@gmail.com>
terryjreedy pushed a commit that referenced this pull request Feb 25, 2024
…115897)

Thanks to Pedro Arthur Duarte (pedroarthur.jedi at gmail.com) for help with this bug.

(cherry picked from commit f745586)

Co-authored-by: Adorilson Bezerra <adorilson@gmail.com>
@adorilson adorilson deleted the improve_datamodel_doc branch February 25, 2024 16:05
woodruffw pushed a commit to woodruffw-forks/cpython that referenced this pull request Mar 4, 2024
Thanks to Pedro Arthur Duarte (pedroarthur.jedi at gmail.com) for help with this bug.
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
Thanks to Pedro Arthur Duarte (pedroarthur.jedi at gmail.com) for help with this bug.
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.

None yet

2 participants