Skip to content

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Oct 10, 2024

(cherry picked from commit a726ce7 -- but only the configparser changes)

Co-authored-by: Alex Waygood Alex.Waygood@Gmail.com
Co-authored-by: Adam Turner 9087854+AA-Turner@users.noreply.github.com


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

…25288)

(cherry picked from commit a726ce7)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
@AA-Turner
Copy link
Member

FileNotFoundError: [Errno 2] No such file or directory: 'my_drawing.ps' - probably because #123617 wasn't backported.

A

Comment on lines 19 to 20
import os
os.remove("my_drawing.ps")
Copy link
Member

Choose a reason for hiding this comment

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

Two birds with one stone; the save feature is new in 3.14, and this works to kill the window after the tests for me locally:

Suggested change
import os
os.remove("my_drawing.ps")
# Destroy the turtle window after tests are complete
# Imported via star import in testsetup
bye()

Copy link
Member

Choose a reason for hiding this comment

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

#125294 (comment) -- this doesn't seem to do anything for me locally :/

for now I'll just revert the changes to turtle.rst and only backport the configparser.rst changes

@AlexWaygood AlexWaygood changed the title [3.13] Add some doctest cleanups for turtle and configparser (GH-125288) [3.13] Add some doctest cleanups for configparser (GH-125288) Oct 11, 2024
@AlexWaygood AlexWaygood enabled auto-merge (squash) October 11, 2024 09:44
@AlexWaygood AlexWaygood merged commit 73c152b into python:3.13 Oct 11, 2024
25 checks passed
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
None yet
Development

Successfully merging this pull request may close these issues.

4 participants