Skip to content

gh-48739: tests(tests_shutil): fix comment with check_args_to_onerror #22968

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Oct 25, 2020

(Note that the special case for Linux/ZFS is not covered [1], and might
actually not work anymore since the changes around handling/setting
self.errorState.)

1: https://codecov.io/gh/python/cpython/src/492d513ccb/Lib/test/test_shutil.py#L354...355

Ref: https://bugs.python.org/issue4489

https://bugs.python.org/issue4489

@github-actions
Copy link

This PR is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 5 days

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Dec 16, 2020
@blueyed
Copy link
Contributor Author

blueyed commented Dec 16, 2020

I think this is still legit, and only changes a comment. Will trigger a rebuild.

- `os.unlink` is used since 2100b42.
- `os.rmdir` is used since 67be92b.

(Note that the special case for Linux/ZFS is not covered [1], and might
actually not work anymore since the changes around handling/setting
`self.errorState`.)

1: https://codecov.io/gh/python/cpython/src/492d513ccb/Lib/test/test_shutil.py#L354...355

Ref: https://bugs.python.org/issue4489
@github-actions github-actions bot removed the stale Stale PR or inactive for long period of time. label Dec 17, 2020
@iritkatriel
Copy link
Member

https://bugs.python.org/issue4489 is closed. What is the status of this PR?

@python-cla-bot
Copy link

python-cla-bot bot commented Apr 6, 2025

The following commit authors need to sign the Contributor License Agreement:

CLA signed

@StanFromIreland StanFromIreland changed the title bpo-4489: tests(tests_shutil): fix comment with check_args_to_onerror gh-48739: tests(tests_shutil): fix comment with check_args_to_onerror Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants