Skip to content

feat(docs): type fix - apply pep8 by using docstring instead of comment in the doc. #135181

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
merged 3 commits into from
Jun 6, 2025

Conversation

jlguenego
Copy link
Contributor

@jlguenego jlguenego commented Jun 5, 2025

replace comments by pep8 docstring for function definition.


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

Giving the right example incitates the tutorial readers to do the same in the future.
@python-cla-bot
Copy link

python-cla-bot bot commented Jun 5, 2025

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app bedevere-app bot added awaiting review docs Documentation in the Doc dir skip news labels Jun 5, 2025
@github-project-automation github-project-automation bot moved this to Todo in Docs PRs Jun 5, 2025
@jlguenego jlguenego changed the title feat(docs): type fix - apply pep8 by using docstring instead of comment in the doc. feat(docs): type fix - apply pep8 by using docstring instead of comment in the doc. skip issue Jun 5, 2025
@jlguenego jlguenego changed the title feat(docs): type fix - apply pep8 by using docstring instead of comment in the doc. skip issue feat(docs): type fix - apply pep8 by using docstring instead of comment in the doc. Jun 5, 2025
Copy link
Contributor

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

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

Since this is the tutorial, I think it makes logical sense to present best practices.

jlguenego and others added 2 commits June 6, 2025 14:37
Co-authored-by: Victor Stinner <vstinner@python.org>
Co-authored-by: Victor Stinner <vstinner@python.org>
@vstinner vstinner merged commit 3431828 into python:main Jun 6, 2025
24 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Docs PRs Jun 6, 2025
@vstinner vstinner added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Jun 6, 2025
@miss-islington-app
Copy link

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

@miss-islington-app
Copy link

Thanks @jlguenego for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 6, 2025
…nt in the doc. (pythonGH-135181)

Giving the right example incitates the tutorial readers to do the same in the future.
(cherry picked from commit 3431828)

Co-authored-by: Jean-Louis GUENEGO <jlguenego@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Jun 6, 2025

GH-135211 is a backport of this pull request to the 3.13 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 6, 2025
…nt in the doc. (pythonGH-135181)

Giving the right example incitates the tutorial readers to do the same in the future.
(cherry picked from commit 3431828)

Co-authored-by: Jean-Louis GUENEGO <jlguenego@gmail.com>
@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jun 6, 2025
@bedevere-app
Copy link

bedevere-app bot commented Jun 6, 2025

GH-135212 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Jun 6, 2025
@vstinner
Copy link
Member

vstinner commented Jun 6, 2025

Merged, thanks.

vstinner pushed a commit that referenced this pull request Jun 6, 2025
…f comment in the doc. (GH-135181) (#135212)

feat(docs): type fix - apply pep8 by using docstring instead of comment in the doc. (GH-135181)

Giving the right example incitates the tutorial readers to do the same in the future.
(cherry picked from commit 3431828)

Co-authored-by: Jean-Louis GUENEGO <jlguenego@gmail.com>
vstinner pushed a commit that referenced this pull request Jun 6, 2025
…f comment in the doc. (GH-135181) (#135211)

feat(docs): type fix - apply pep8 by using docstring instead of comment in the doc. (GH-135181)

Giving the right example incitates the tutorial readers to do the same in the future.
(cherry picked from commit 3431828)

Co-authored-by: Jean-Louis GUENEGO <jlguenego@gmail.com>
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
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants