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.

Sign up for GitHub

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-91317: Document that Path does not collapse initial // #32193

Merged
merged 10 commits into from
Jun 10, 2022
Merged

gh-91317: Document that Path does not collapse initial // #32193

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
fb49937
Document that Path does not collapse initial `//`
arhadthedev Mar 30, 2022
0cf996b
Add a NEWS entry
arhadthedev Mar 30, 2022
ad0de01
Update Misc/NEWS.d/next/Documentation/2022-03-30-17-56-01.bpo-47161.g…
arhadthedev Mar 31, 2022
1ed30e6
Merge branch 'main' into bpo-47161
arhadthedev Jun 8, 2022
ab2f285
Address the review of Bratt Cannon
arhadthedev Jun 8, 2022
31dc415
Address the review of Barney Gale
arhadthedev Jun 8, 2022
207b2d6
Do not use spaces inside links
arhadthedev Jun 8, 2022
519078a
Address "Unknown target name: unc"
arhadthedev Jun 8, 2022
a307bd3
Fix the link
arhadthedev Jun 8, 2022
6d76a31
Remove my provenance from NEWS because it's a community effort now.
arhadthedev Jun 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions Doc/library/pathlib.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,11 +133,14 @@ we also call *flavours*:
PureWindowsPath('c:/Program Files')

Spurious slashes and single dots are collapsed, but double dots (``'..'``)
are not, since this would change the meaning of a path in the face of
symbolic links::
and initial double slashes (``'//'``) are not, since this would change the
meaning of a path in the face of symbolic links and UNC (network) paths
respectively::
arhadthedev marked this conversation as resolved.
Show resolved Hide resolved

>>> PurePath('foo//bar')
PurePosixPath('foo/bar')
>>> PurePath('//foo/bar')
PurePosixPath('//foo/bar')
>>> PurePath('foo/./bar')
PurePosixPath('foo/bar')
>>> PurePath('foo/../bar')
Expand Down
2 changes: 2 additions & 0 deletions Misc/NEWS.d/next/Documentation/2022-03-30-17-56-01.bpo-47161.gesHfS.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Explicitly documented that :class:`pathlib.PurePath` does not collapse
arhadthedev marked this conversation as resolved.
Show resolved Hide resolved
initial double slashes because they denote UNC paths. Patch by Oleg Iarygin.