Skip to content

GH-128520: pathlib ABCs: validate magic_open() arguments #131617

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 1 commit into from
Mar 24, 2025

Conversation

barneygale
Copy link
Contributor

@barneygale barneygale commented Mar 23, 2025

When pathlib._os.magic_open() is called to open a path in binary mode, raise ValueError if any of the encoding, errors or newline arguments are given. This matches the open() built-in.

When `pathlib._os.magic_open()` is called to open a path in binary mode,
raise `ValueError` if any of the *encoding*, *errors* or *newline*
arguments are given. This matches the `open()` built-in.
@barneygale barneygale merged commit d716ea3 into python:main Mar 24, 2025
44 checks passed
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 1, 2025
…hon#131617)

When `pathlib._os.magic_open()` is called to open a path in binary mode,
raise `ValueError` if any of the *encoding*, *errors* or *newline*
arguments are given. This matches the `open()` built-in.
seehwan pushed a commit to seehwan/cpython that referenced this pull request Apr 16, 2025
…hon#131617)

When `pathlib._os.magic_open()` is called to open a path in binary mode,
raise `ValueError` if any of the *encoding*, *errors* or *newline*
arguments are given. This matches the `open()` built-in.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant