Skip to content

Conversation

barneygale
Copy link
Contributor

@barneygale barneygale commented Nov 29, 2024

Call urllib.request.pathname2url() from pathlib.Path.as_uri(), and deprecate the duplicate implementation in PurePath.


📚 Documentation preview 📚: https://cpython-previews--127380.org.readthedocs.build/

Call `urllib.request.pathname2url()` from `pathlib.Path.as_uri()`, and
deprecate the duplicate implementation in `PurePath`.
Copy link
Member

@AA-Turner AA-Turner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Sorry for missing this PR initially, I only spotted it when you closed the other PR earlier today.

A

@barneygale
Copy link
Contributor Author

barneygale commented Mar 20, 2025

No worries, and thanks very much for the review!

Embarrassingly I actually linked the wrong PR from my comment in #123650 😅. I've edited it now.

@barneygale barneygale merged commit f141e8e into python:main Mar 20, 2025
43 checks passed
@AA-Turner
Copy link
Member

Embarrassingly I actually linked the wrong PR from my comment in #123650 😅.

Et voilà: #126844 (review)

A

seehwan pushed a commit to seehwan/cpython that referenced this pull request Apr 16, 2025
…ython#127380)

Call `urllib.request.pathname2url()` from `pathlib.Path.as_uri()`, and
deprecate the duplicate implementation in `PurePath`.

Co-authored-by: Adam Turner <[email protected]>
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.

3 participants