Skip to content

Make pygame.typing.PathLike internal/private - #3086

Merged
damusss merged 3 commits into
pygame-community:mainfrom
aatle:change-pathlike
Aug 31, 2024
Merged

damusss merged 3 commits into
pygame-community:mainfrom
aatle:change-pathlike

Conversation

@aatle

@aatle aatle commented Aug 30, 2024

Copy link
Copy Markdown
Member

Closes #3085

Renames all occurrences from PathLike to _PathLike, and removes it from docs and __all__ list.

@aatle
aatle requested a review from a team as a code owner August 30, 2024 01:05

@damusss damusss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This isn't particularly important, the rest looks good!

Comment thread docs/reST/ref/typing.rst Resolved

@MyreMylar MyreMylar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This seems reasonable I agree that the the os.PathLike vs pygame.typing.PathLike is likely to be confusing.

@bilhox bilhox added the type hints Type hint checking related tasks label Aug 30, 2024

@ankith26 ankith26 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks for the PR 🎉

@ankith26 ankith26 added this to the 2.5.2 milestone Aug 31, 2024

@damusss damusss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, Thank you!

@damusss
damusss merged commit b0d45d1 into pygame-community:main Aug 31, 2024
@aatle
aatle deleted the change-pathlike branch August 31, 2024 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type hints Type hint checking related tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal: Remove pygame.typing.PathLike type from the API

5 participants