-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
Fix print
usage in turtle
doctests
#122940
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
Conversation
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.
Are the turtle docstests running? As surely they should have been failing without this change...
A
Nope, they are not real doctests, they just look like doctests. They are called examples in most cases. I think that this is fine, users can still benefit from it. And I guess converting them to real doctests might be really hard. |
Thanks @sobolevn for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13. |
(cherry picked from commit 901d949) Co-authored-by: sobolevn <[email protected]>
GH-122977 is a backport of this pull request to the 3.13 branch. |
(cherry picked from commit 901d949) Co-authored-by: sobolevn <[email protected]>
GH-122978 is a backport of this pull request to the 3.12 branch. |
Fix `print` usage in `turtle` doctests (GH-122940) (cherry picked from commit 901d949) Co-authored-by: sobolevn <[email protected]>
Fix `print` usage in `turtle` doctests (GH-122940) (cherry picked from commit 901d949) Co-authored-by: sobolevn <[email protected]>
No description provided.