Skip to content

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

Open
blueyed wants to merge 1 commit into
python: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 into
python:mainfrom
blueyed:test-fix-comment

Conversation

@blueyed

@blueyed blueyed commented Oct 25, 2020

Copy link
Copy Markdown
Contributor

(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

blueyed commented Dec 16, 2020

Copy link
Copy Markdown
Contributor Author

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

python-cla-bot Bot commented Apr 6, 2025

Copy link
Copy Markdown

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

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown

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