Skip to content

Conversation

alex-semenyuk
Copy link
Contributor

@alex-semenyuk alex-semenyuk commented Jul 19, 2025

Add TurtleGraphicsError documentation


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

@StanFromIreland
Copy link
Member

PS @willingc (per devguide, the expert) you are not a codeowner of the docs file.

@alex-semenyuk
Copy link
Contributor Author

@willingc friendly ping

Copy link
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 you for the PR, @alex-semenyuk.

I think it is too long.

I suggest the changes:

  1. For definition, keeping only the first sentence
  2. Adding that this exception extends Exception
  3. Shortening the examples. One or two raises in a single doctest session should be sufficient. As a consequence, removing the Invalid...* session titles.

You can look for the Exceptions session in other modules to get the feeling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core review docs Documentation in the Doc dir skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

3 participants