Skip to content

Conversation

@hridyasadanand
Copy link

@hridyasadanandhridyasadanand commented Mar 29, 2025

Added cross-references to related functions and provided an example for os.path.join to enhance clarity.


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

Added cross-references to related functions and provided an example for os.path.join to enhance clarity.
@ghost
Copy link

ghost commented Mar 29, 2025

All commit authors signed the Contributor License Agreement.
CLA signed

Copy link
Member

@StanFromIrelandStanFromIreland left a comment

Choose a reason for hiding this comment

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

Very messy

Return ``True`` if *path* is an absolute pathname.

.. seealso::
:func:`os.path.abspath` - for converting paths to absolute paths. On Unix, that means it

Choose a reason for hiding this comment

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

This is a mess

Comment on lines +247 to +248
.. seealso::
:func:`os.path.abspath` - for converting paths to absolute paths. On Unix, that means it

Choose a reason for hiding this comment

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

Usually these are put at the end of the note

Comment on lines +367 to 368
'/home/bar' The return value is the
concatenation of *path* and all members of *\*paths*, with exactly one

Choose a reason for hiding this comment

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

Needs to be fixed

Comment on lines +506 to 507
:func:`os.path.join` - for joining paths efficiently. The
*tail* part will never contain a slash; if *path* ends in a slash, *tail*

Choose a reason for hiding this comment

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

Again a mess

@picnixz
Copy link
Member

#130557 already exists. Please do not open PRs when an open PR against the issue already exists. TiA.

@picnixzpicnixz closed this Mar 29, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting reviewdocsDocumentation in the Doc dirskip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

3 participants

@hridyasadanand@picnixz@StanFromIreland