Skip to content

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

Open
blueyed wants to merge 1 commit intopython:mainfrom
blueyed:test-fix-comment
Open

gh-48739: tests(tests_shutil): fix comment with check_args_to_onerror#22968
blueyed wants to merge 1 commit intopython:mainfrom
blueyed:test-fix-comment

Conversation

@blueyed
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
Member

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

@python-cla-bot
Copy link
Copy Markdown

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
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review stale Stale PR or inactive for long period of time. tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants