Skip to content

bpo-42999: expand and clarify pathlib.Path.link_to() documentation.#24294

Merged
zooba merged 1 commit intopython:masterfrom
barneygale:bpo-42999-pathlib-link-to-docs
Apr 7, 2021
Merged

bpo-42999: expand and clarify pathlib.Path.link_to() documentation.#24294
zooba merged 1 commit intopython:masterfrom
barneygale:bpo-42999-pathlib-link-to-docs

Conversation

@barneygale
Copy link
Copy Markdown
Contributor

@barneygale barneygale commented Jan 22, 2021 •
edited
Loading

  • Replace:

    Create a hard link pointing to a path named target.

    With:

    Make target a hard link to this path.

  • Add warning that the argument order is reversed from what's expected.

  • Move link_to() directly under symlink_to() for ease of comparison (docs + code)

See also: https://bugs.python.org/issue39950

https://bugs.python.org/issue42999

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 22, 2021

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Feb 22, 2021
@escape0707
Copy link
Copy Markdown

escape0707 commented Apr 7, 2021 •
edited
Loading

Wow, this issue still doesn't get addressed after so many months. So when can we even progress to the next step to fix this historical problem and implement a correct version of this function...

More days we wait, more compromises caused by this prolonging historical mistake will take place in downstream developments.

@barneygale
Copy link
Copy Markdown
Contributor Author

barneygale commented Apr 7, 2021

Wow, this issue still doesn't get addressed after so many months. So when can we even progress to the next step to fix this historical problem and implement a correct version of this function...

More days we wait, more compromises caused by this prolonging historical mistake will take place in downstream developments.

I haven't been particularly vocal about getting this reviewed (IRL commitments), so I wouldn't take it as indicative of Python development in general.

@barneygale barneygale force-pushed the bpo-42999-pathlib-link-to-docs branch from d59debe to f7640fd Compare April 7, 2021 13:58
@zooba
Copy link
Copy Markdown
Member

zooba commented Apr 7, 2021

Didn't I merge this one yesterday? I was sure I did...

@zooba zooba merged commit 8aac1be into python:master Apr 7, 2021
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Apr 7, 2021

Thanks @barneygale for the PR, and @zooba for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8, 3.9.
🐍🍒⛏🤖

@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Apr 7, 2021

Sorry, @barneygale and @zooba, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 8aac1bea2eeac25a49f8912b67aacbedf9bc7934 3.9

@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Apr 7, 2021

Sorry @barneygale and @zooba, I had trouble checking out the 3.8 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 8aac1bea2eeac25a49f8912b67aacbedf9bc7934 3.8

zooba pushed a commit to zooba/cpython that referenced this pull request Apr 7, 2021
@barneygale @zooba
bpo-42999: Expand and clarify pathlib.Path.link_to() documentation. (p…
fee60e7 
…ythonGH-24294)
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Apr 7, 2021

GH-25257 is a backport of this pull request to the 3.9 branch.

zooba added a commit that referenced this pull request Apr 7, 2021
@zooba @barneygale
bpo-42999: Expand and clarify pathlib.Path.link_to() documentation. (G…
34f9300 
…H-24294)

Co-authored-by: Barney Gale <barney.gale@gmail.com>
zooba added a commit to zooba/cpython that referenced this pull request Apr 7, 2021
@zooba @barneygale
bpo-42999: Expand and clarify pathlib.Path.link_to() documentation. (p…
62bdf02 
…ythonGH-24294)

Co-authored-by: Barney Gale <barney.gale@gmail.com>
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Apr 7, 2021

GH-25258 is a backport of this pull request to the 3.8 branch.

zooba added a commit that referenced this pull request Apr 7, 2021
@zooba @barneygale
bpo-42999: Expand and clarify pathlib.Path.link_to() documentation. (G…
9f57873 
…H-24294)

Co-authored-by: Barney Gale <barney.gale@gmail.com>
@alexdelorenzo alexdelorenzo mentioned this pull request Apr 16, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zooba zooba zooba approved these changes

Assignees

@zooba zooba

Labels

skip news stale Stale PR or inactive for long period of time.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@barneygale @escape0707 @zooba @miss-islington @bedevere-bot @the-knights-who-say-ni