Skip to content

gh-119671: Add os.path.pure #119772

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.

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

Closed
wants to merge 19 commits into from
Closed

Conversation

nineteendo
Copy link
Contributor

@nineteendo nineteendo commented May 30, 2024

@nineteendo nineteendo changed the title [WIP] Add os.path.pure [WIP] gh-119671: Add os.path.pure May 30, 2024
@nineteendo
Copy link
Contributor Author

nineteendo commented May 30, 2024

@barneygale, it could look something like this.

@nineteendo nineteendo changed the title [WIP] gh-119671: Add os.path.pure gh-119671: Add os.path.pure Jun 13, 2024
@nineteendo nineteendo marked this pull request as ready for review June 13, 2024 10:58
@nineteendo nineteendo requested review from a team and ericsnowcurrently as code owners June 13, 2024 10:58
@nineteendo
Copy link
Contributor Author

nineteendo commented Jun 13, 2024

@barneygale, all tests are passing now. I made os.path.pure simply export all safe functions from os.path.
When this is merged, the safe functions could be moved from os.path to os.path.pure.

Should defpath & devnull be exported?

@nineteendo
Copy link
Contributor Author

Thanks Erlend.

@nineteendo nineteendo closed this Oct 8, 2024
@nineteendo nineteendo deleted the add-os.path.pure branch October 8, 2024 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants