Skip to content

Conversation

@alex-semenyuk
Copy link
Contributor

@alex-semenyuk alex-semenyuk commented Jun 15, 2025

Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

Thanks!

@zooba or @chris-eibl: As Windows experts, would either of you like to look at this one?

Copy link
Member

@chris-eibl chris-eibl left a comment

Choose a reason for hiding this comment

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

LGTM. A link to an example is always a good thing.

@zooba
Copy link
Member

zooba commented Jun 17, 2025

It's not particularly a Windows-specific example, so it seems like mentioning that might put people off reading it. It's a good example of using the error callback.

Also I think we tend to use just "See ..." for examples, rather than "See Also" callouts. But that's a question for the docs WG.

@alex-semenyuk alex-semenyuk force-pushed the link_to_shutil-rmtree-example branch from 96a60bb to 1bd0a48 Compare June 18, 2025 20:28
@alex-semenyuk
Copy link
Contributor Author

Agree. Removed Windows mention

@zooba zooba merged commit e9b647d into python:main Jun 18, 2025
24 checks passed
@miss-islington-app
Copy link

Thanks @alex-semenyuk for the PR, and @zooba for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

@github-project-automation github-project-automation bot moved this from Todo to Done in Docs PRs Jun 18, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 18, 2025
@bedevere-app
Copy link

bedevere-app bot commented Jun 18, 2025

GH-135691 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Jun 18, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 18, 2025
@bedevere-app
Copy link

bedevere-app bot commented Jun 18, 2025

GH-135692 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jun 18, 2025
zooba pushed a commit that referenced this pull request Jun 18, 2025
)

(cherry picked from commit e9b647d)

Co-authored-by: alexey semenyuk <[email protected]>
zooba pushed a commit that referenced this pull request Jun 18, 2025
)

(cherry picked from commit e9b647d)

Co-authored-by: alexey semenyuk <[email protected]>
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 news

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants