GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-100783: fix os.path.join documentation by hauntsaninja · Pull Request #100811 · python/cpython

Conversation

@hauntsaninja

@hauntsaninja hauntsaninja commented Jan 6, 2023 •
edited by bedevere-bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor
  • Use "drive", not "drive letter", because of UNC paths
  • Previous components are not thrown away from relative drive letters
  • Use "segment" instead of "component" for consistency with pathlib

See issue for more details

  • Issue: Clarify os.path.join documentation #100783

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@hauntsaninja
pythongh-100783: fix os.path.join documentation
ccb103f
- Use "drive", not "drive letter", because of UNC paths
- Previous components are not thrown away from relative drive letters
- Use "segment" instead of "component" for consistency with pathlib

See issue for more details
@bedevere-bot bedevere-bot added the awaiting core review label Jan 6, 2023
@bedevere-bot bedevere-bot mentioned this pull request Jan 6, 2023
Clarify os.path.join documentation #100783
Closed
@bedevere-bot bedevere-bot added docs Documentation in the Doc dir skip news labels Jan 6, 2023
@hauntsaninja
hauntsaninja requested review from JelleZijlstra and barneygale January 6, 2023 21:02
@hauntsaninja hauntsaninja added needs backport to 3.10 needs backport to 3.11 only security fixes labels Jan 6, 2023
eryksun
eryksun reviewed Jan 6, 2023
View reviewed changes
Comment thread Doc/library/os.path.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

eryksun
eryksun reviewed Jan 6, 2023
View reviewed changes
Comment thread Doc/library/os.path.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

hauntsaninja added 2 commits January 6, 2023 14:44
@hauntsaninja
clarify "absolute"
5a3589b
@hauntsaninja
change preposition
8c3acbf
@hauntsaninja

hauntsaninja commented Jan 6, 2023

Copy link
Copy Markdown
Contributor Author

Thanks for the review, made the changes!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

JelleZijlstra
JelleZijlstra approved these changes Jan 7, 2023
View reviewed changes

@JelleZijlstra JelleZijlstra left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Looks good, one nit in nearby code.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread Doc/library/os.path.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot bedevere-bot added awaiting merge and removed awaiting core review labels Jan 7, 2023
@hauntsaninja
Update Doc/library/os.path.rst
a2a6c81
hauntsaninja
hauntsaninja commented Jan 7, 2023
View reviewed changes
Comment thread Doc/library/os.path.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@hauntsaninja
Update Doc/library/os.path.rst
4729442
eryksun
eryksun reviewed Jan 8, 2023
View reviewed changes
Comment thread Doc/library/os.path.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@hauntsaninja
split sentences
403ef6e
@hauntsaninja
hauntsaninja merged commit 53455a3 into python:main Jan 8, 2023
@miss-islington

miss-islington commented Jan 8, 2023

Copy link
Copy Markdown
Contributor

Thanks @hauntsaninja for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot bedevere-bot removed the awaiting merge label Jan 8, 2023
@hauntsaninja
hauntsaninja deleted the gh-100783 branch January 8, 2023 08:51
@bedevere-bot

bedevere-bot commented Jan 8, 2023

Copy link
Copy Markdown

GH-100843 is a backport of this pull request to the 3.11 branch.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Jan 8, 2023
@bedevere-bot

bedevere-bot commented Jan 8, 2023

Copy link
Copy Markdown

GH-100844 is a backport of this pull request to the 3.10 branch.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 label Jan 8, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 8, 2023
@hauntsaninja @miss-islington
pythongh-100783: fix os.path.join documentation (pythonGH-100811)
6860d99
- Use "drive", not "drive letter", because of UNC paths
- Previous components are not thrown away from relative drive letters
- Use "segment" instead of "component" for consistency with pathlib
- Other miscellaneous improvements
(cherry picked from commit 53455a3)

Co-authored-by: Shantanu <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 8, 2023
@hauntsaninja @miss-islington
pythongh-100783: fix os.path.join documentation (pythonGH-100811)
baefa55
- Use "drive", not "drive letter", because of UNC paths
- Previous components are not thrown away from relative drive letters
- Use "segment" instead of "component" for consistency with pathlib
- Other miscellaneous improvements
(cherry picked from commit 53455a3)

Co-authored-by: Shantanu <[email protected]>
miss-islington added a commit that referenced this pull request Jan 8, 2023
@miss-islington @hauntsaninja
gh-100783: fix os.path.join documentation (GH-100811)
87076d8
- Use "drive", not "drive letter", because of UNC paths
- Previous components are not thrown away from relative drive letters
- Use "segment" instead of "component" for consistency with pathlib
- Other miscellaneous improvements
(cherry picked from commit 53455a3)

Co-authored-by: Shantanu <[email protected]>
miss-islington added a commit that referenced this pull request Jan 8, 2023
@miss-islington @hauntsaninja
gh-100783: fix os.path.join documentation (GH-100811)
fa8d396
- Use "drive", not "drive letter", because of UNC paths
- Previous components are not thrown away from relative drive letters
- Use "segment" instead of "component" for consistency with pathlib
- Other miscellaneous improvements
(cherry picked from commit 53455a3)

Co-authored-by: Shantanu <[email protected]>
hauntsaninja added a commit to hauntsaninja/cpython that referenced this pull request Jan 29, 2023
@hauntsaninja
pythongh-77607: Improve accuracy of os.path.join docs
392c285
This is a follow-up to python#100811.

One of the changes in that PR isn't accurate in that
`os.path.join('', '')` will not end in a separator.

This reverts that change to the previous wording that used "only", but
explicitly calls out the case where the last part ends in a separator,
which is what caused confusin in python#77607 and motivated the change
in python#100811.
@hauntsaninja hauntsaninja mentioned this pull request Jan 29, 2023
gh-77607: Improve accuracy of os.path.join docs #101406
Merged
hauntsaninja added a commit that referenced this pull request Jan 31, 2023
@hauntsaninja
gh-77607: Improve accuracy of os.path.join docs (#101406)
909a674
This is a follow-up to #100811.

One of the changes in that PR isn't accurate in that
`os.path.join('', '')` will not end in a separator.

This reverts that change to the previous wording that used "only", but
explicitly calls out the case where the last part ends in a separator,
which is what caused confusin in #77607 and motivated the change
in #100811.
@miss-islington miss-islington mentioned this pull request Jan 31, 2023
[3.11] gh-77607: Improve accuracy of os.path.join docs (GH-101406) #101451
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 31, 2023
@hauntsaninja @miss-islington
pythongh-77607: Improve accuracy of os.path.join docs (pythonGH-101406)
51b00e2
This is a follow-up to pythonGH-100811.

One of the changes in that PR isn't accurate in that
`os.path.join('', '')` will not end in a separator.

This reverts that change to the previous wording that used "only", but
explicitly calls out the case where the last part ends in a separator,
which is what caused confusin in pythonGH-77607 and motivated the change
in pythonGH-100811.
(cherry picked from commit 909a674)

Co-authored-by: Shantanu <[email protected]>
@miss-islington miss-islington mentioned this pull request Jan 31, 2023
[3.10] gh-77607: Improve accuracy of os.path.join docs (GH-101406) #101452
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 31, 2023
@hauntsaninja @miss-islington
pythongh-77607: Improve accuracy of os.path.join docs (pythonGH-101406)
46a47ac
This is a follow-up to pythonGH-100811.

One of the changes in that PR isn't accurate in that
`os.path.join('', '')` will not end in a separator.

This reverts that change to the previous wording that used "only", but
explicitly calls out the case where the last part ends in a separator,
which is what caused confusin in pythonGH-77607 and motivated the change
in pythonGH-100811.
(cherry picked from commit 909a674)

Co-authored-by: Shantanu <[email protected]>
miss-islington added a commit that referenced this pull request Jan 31, 2023
@miss-islington @hauntsaninja
gh-77607: Improve accuracy of os.path.join docs (GH-101406)
f36c272
This is a follow-up to GH-100811.

One of the changes in that PR isn't accurate in that
`os.path.join('', '')` will not end in a separator.

This reverts that change to the previous wording that used "only", but
explicitly calls out the case where the last part ends in a separator,
which is what caused confusin in GH-77607 and motivated the change
in GH-100811.
(cherry picked from commit 909a674)

Co-authored-by: Shantanu <[email protected]>
miss-islington added a commit that referenced this pull request Jan 31, 2023
@miss-islington @hauntsaninja
gh-77607: Improve accuracy of os.path.join docs (GH-101406)
a79dd88
This is a follow-up to GH-100811.

One of the changes in that PR isn't accurate in that
`os.path.join('', '')` will not end in a separator.

This reverts that change to the previous wording that used "only", but
explicitly calls out the case where the last part ends in a separator,
which is what caused confusin in GH-77607 and motivated the change
in GH-100811.
(cherry picked from commit 909a674)

Co-authored-by: Shantanu <[email protected]>
mdboom pushed a commit to mdboom/cpython that referenced this pull request Jan 31, 2023
@hauntsaninja @mdboom
pythongh-77607: Improve accuracy of os.path.join docs (python#101406)
1a51ee1
This is a follow-up to python#100811.

One of the changes in that PR isn't accurate in that
`os.path.join('', '')` will not end in a separator.

This reverts that change to the previous wording that used "only", but
explicitly calls out the case where the last part ends in a separator,
which is what caused confusin in python#77607 and motivated the change
in python#100811.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eryksun eryksun eryksun left review comments
@JelleZijlstra JelleZijlstra JelleZijlstra approved these changes
@barneygale barneygale Awaiting requested review from barneygale

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@hauntsaninja @miss-islington @bedevere-bot @JelleZijlstra @eryksun
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.