Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Open
blueyed wants to merge 1 commit intopython:mainpython/cpython:mainfrom
blueyed:test-fix-commentblueyed/cpython:test-fix-commentCopy head branch name to clipboard
Open

gh-48739: tests(tests_shutil): fix comment with check_args_to_onerror#22968
blueyed wants to merge 1 commit intopython:mainpython/cpython:mainfrom
blueyed:test-fix-commentblueyed/cpython:test-fix-commentCopy head branch name to clipboard

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.