Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-134538: link to example for windows readonly file handling in function docs #135540

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

Merged
merged 2 commits into from
Jun 18, 2025

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