Skip to content
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

GH-110109: pathlib ABCs: drop use of warnings._deprecated() #113419

Merged
merged 2 commits into from
Dec 27, 2023

Conversation

barneygale
Copy link
Contributor

@barneygale barneygale commented Dec 23, 2023

The pathlib._abc module will be made available as a PyPI backport supporting Python 3.8+. The warnings._deprecated() function was only added last year, and it's private from an external package perspective, so here we switch to warnings.warn() instead.

The `pathlib._abc` module will be made available as a PyPI backport
supporting Python 3.8+. The `warnings._deprecated()` function was only
added last year, and it's private from an external package perspective, so
here we switch to `warnings.warn()` instead.
@barneygale
Copy link
Contributor Author

Downstream pathlib_abc issue: barneygale/pathlib-abc#11

@barneygale barneygale merged commit 1b19d73 into python:main Dec 27, 2023
26 checks passed
kulikjak pushed a commit to kulikjak/cpython that referenced this pull request Jan 22, 2024
…ython#113419)

The `pathlib._abc` module will be made available as a PyPI backport
supporting Python 3.8+. The `warnings._deprecated()` function was only
added last year, and it's private from an external package perspective, so
here we switch to `warnings.warn()` instead.
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…ython#113419)

The `pathlib._abc` module will be made available as a PyPI backport
supporting Python 3.8+. The `warnings._deprecated()` function was only
added last year, and it's private from an external package perspective, so
here we switch to `warnings.warn()` instead.
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.

None yet

1 participant