GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-157847: Re-write more of the `turtle` reference docs by StanFromIreland · Pull Request #159685 · python/cpython

Conversation

@StanFromIreland

@StanFromIreland StanFromIreland commented Oct 18, 2026

Copy link
Copy Markdown
Member

No description provided.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
m-aciek reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@StanFromIreland
Re-write more turtle docs
eff9d13
@StanFromIreland
StanFromIreland requested review from m-aciek and willingc October 18, 2026 19:51
@StanFromIreland StanFromIreland added docs Documentation in the Doc dir skip issue skip news labels Oct 18, 2026
@github-project-automation github-project-automation Bot added this to Docs PRs Oct 18, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Oct 18, 2026
@bedevere-app bedevere-app Bot added the awaiting core review label Oct 18, 2026
@read-the-docs-community

read-the-docs-community Bot commented Oct 18, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #35277420 | 📁 Comparing 284e767 against main (7dbc797)

  🔍 Preview build  

18 files changed · ± 18 modified

± Modified

  • py-modindex.html
  • c-api/extension-modules.html
  • c-api/file.html
  • c-api/import.html
  • c-api/interp-lifecycle.html
  • c-api/intro.html
  • c-api/structures.html
  • library/bdb.html
  • library/ctypes.html
  • library/gc.html
  • and 8 more...
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
willingc
willingc approved these changes Oct 18, 2026
View reviewed changes

@willingc willingc left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Nice cleanup @StanFromIreland. A few suggestions for readability for children and young adults. Fine to ship as is too.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread Doc/library/turtle.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/library/turtle.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/library/turtle.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Lib/turtle.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Lib/turtle.py
9
"""
speeds = {'fastest':0, 'fast':10, 'normal':6, 'slow':3, 'slowest':1 }
speed_strings = {

@willingc willingc Oct 18, 2026

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Is there a reason to name it speed_strings vs. speeds?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@StanFromIreland StanFromIreland Oct 20, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

I was going to tidy up the dictionary (since it was in a bit of a mess), and I decided to change to a more precise name while I was at it. As I mentioned on the other PRs, I'm not particularly worried about churn, most of these lines blame to the original import anyway.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@willingc willingc Oct 20, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Can we do this... Add the word "descriptive" at the start of line 2219 so it reads "descriptive speed string" or alternatively "speed description string". speed_strings is a bit confusing for learners since it encodes type in the name of the dict.

If we describe a bit more precisely at line 2219, the example name makes more sense.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@StanFromIreland StanFromIreland Oct 21, 2026

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Hmm, I think you may have been looking at an older commit? I adjusted it to:

You can also use these names instead of numbers:

Based on Maciej's suggestion.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread Lib/turtle.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot added awaiting merge and removed awaiting core review labels Oct 18, 2026
m-aciek
m-aciek approved these changes Oct 19, 2026
View reviewed changes
Comment thread Doc/library/turtle.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/library/turtle.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/library/turtle.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/library/turtle.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Lib/turtle.py Outdated
Comment on lines +2179 to +2183
"""Pull the pen up.

Aliases: penup | pu | up

The turtle does not draw when it moves.

@m-aciek m-aciek Oct 19, 2026

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Suggested change
"""Pull the pen up.
Aliases: penup | pu | up
The turtle does not draw when it moves.
"""Lift the pen. The turtle does not draw when it moves.
Aliases: penup | pu | up

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@StanFromIreland StanFromIreland Oct 20, 2026

Copy link
Copy Markdown
Member Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Applied the re-wording, but I kept the other sentence after the aliases.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@m-aciek m-aciek Oct 20, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

I think they shouldn't be splitted. The effect should be an inherent part of the explanation.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
StanFromIreland reacted with eyes emoji
All reactions
  • 👀 1 reaction
Comment thread Lib/turtle.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
@StanFromIreland @m-aciek @willingc
Apply review suggestions
284e767 
Co-authored-by: Maciej Olko <[email protected]>
Co-Authored-By: Carol Willing <[email protected]>
@StanFromIreland
StanFromIreland enabled auto-merge (squash) October 20, 2026 18:21
@StanFromIreland StanFromIreland added needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes labels Oct 20, 2026
@StanFromIreland
StanFromIreland merged commit f071bac into python:main Oct 20, 2026
57 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@StanFromIreland
StanFromIreland deleted the turtle/another-round branch October 20, 2026 19:20
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Oct 20, 2026
@miss-islington-app

miss-islington-app Bot commented Oct 20, 2026

Copy link
Copy Markdown

Thanks @StanFromIreland for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14, 3.15.
🐍🍒⛏🤖
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the awaiting merge label Oct 20, 2026
@bedevere-app

bedevere-app Bot commented Oct 20, 2026

Copy link
Copy Markdown

GH-159948 is a backport of this pull request to the 3.15 branch.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the needs backport to 3.15 bugs and security fixes label Oct 20, 2026
@bedevere-app

bedevere-app Bot commented Oct 20, 2026

Copy link
Copy Markdown

GH-159949 is a backport of this pull request to the 3.14 branch.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Oct 20, 2026
JelleZijlstra pushed a commit that referenced this pull request Oct 21, 2026
@miss-islington @StanFromIreland
@m-aciek @willingc
[3.14] gh-157847: Re-write more of the turtle reference docs (GH-15…
a3f95bb 
…9685) (#159949)

gh-157847: Re-write more of the `turtle` reference docs (GH-159685)
(cherry picked from commit f071bac)

Co-authored-by: Stan Ulbrych <[email protected]>
Co-authored-by: Maciej Olko <[email protected]>
Co-authored-by: Carol Willing <[email protected]>
JelleZijlstra pushed a commit that referenced this pull request Oct 21, 2026
@miss-islington @StanFromIreland
@m-aciek @willingc
[3.15] gh-157847: Re-write more of the turtle reference docs (GH-15…
815057b 
…9685) (#159948)

gh-157847: Re-write more of the `turtle` reference docs (GH-159685)
(cherry picked from commit f071bac)

Co-authored-by: Stan Ulbrych <[email protected]>
Co-authored-by: Maciej Olko <[email protected]>
Co-authored-by: Carol Willing <[email protected]>
@StanFromIreland

StanFromIreland commented Oct 21, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

Sorry @willingc and @m-aciek, I enabled auto-merge and it merged before I saw your comments (feel free to disable it in the future, Maciek, you have to close and reopen the PR IIRC).

I will address them in the next PR (as I still have a bit to do).
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@willingc

willingc commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

No worries @StanFromIreland. It was a nit more than critical since I had already approved 😄
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@willingc willingc willingc approved these changes
@m-aciek m-aciek m-aciek approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@StanFromIreland @willingc @m-aciek
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.