Skip to content

gh-138845: Correct turtle module documentation misconceptions#138882

Open
F0RRZZ wants to merge 4 commits intopython:mainfrom
F0RRZZ:issue-138845
Open

gh-138845: Correct turtle module documentation misconceptions#138882
F0RRZZ wants to merge 4 commits intopython:mainfrom
F0RRZZ:issue-138845

Conversation

@F0RRZZ
Copy link
Copy Markdown
Contributor

@F0RRZZ F0RRZZ commented Sep 14, 2025

@bedevere-app

This comment was marked as resolved.

Copy link
Copy Markdown
Contributor

@adorilson adorilson left a comment

Choose a reason for hiding this comment

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

Thank for you PR, @F0RRZZ

I made some comments.

Comment thread Doc/library/turtle.rst Outdated
variables to their initial values:

* position: ``(0, 0)``
* heading: ``0`` (east)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe

  • heading: 0 (east if standard or world mode, north if logo mode)

Comment thread Doc/library/turtle.rst Outdated
Comment on lines +1399 to +1401
Delete the turtle's drawings from the screen. Move the turtle
to position ``(0, 0)``, set its heading to the default
(facing east in standard mode) and restore the following state
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think these parentheses are unnecessary.

Maybe, you could say

set its heading to 0 and...

Comment thread Lib/turtle.py Outdated
Comment on lines +2648 to +2654
Delete the turtle's drawings from the screen. Move the turtle
to position `(0, 0)`, set its heading to the default
(facing east in standard mode) and restore the following state
variables to their initial values:

- position: `(0, 0)`
- heading: `0` (east)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The same thing concerns heading here.

@F0RRZZ
Copy link
Copy Markdown
Contributor Author

F0RRZZ commented Sep 21, 2025

@adorilson Fixed, thank you for the suggestions!

@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review skip news stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants