Skip to content
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

Replace os.path with pathlib #1834

Closed
seisman opened this issue Mar 19, 2022 · 2 comments · Fixed by #3119 or #3129
Closed

Replace os.path with pathlib #1834

seisman opened this issue Mar 19, 2022 · 2 comments · Fixed by #3119 or #3129
Assignees
Labels
maintenance Boring but important stuff for the core devs
Milestone

Comments

@seisman
Copy link
Member

seisman commented Mar 19, 2022

pathlib is the standard and preferred way to manipulate paths. We should migrate os.path calls to pathlib.

References:

@seisman seisman added the maintenance Boring but important stuff for the core devs label Mar 19, 2022
@seisman
Copy link
Member Author

seisman commented May 12, 2022

Migrating from os.path to pathlib.Path also help testing the pathlib support of PyGMT, but before making the changes, we need to address #1912 first.

@seisman seisman added this to the 0.8.0 milestone Dec 10, 2022
@seisman seisman self-assigned this Dec 10, 2022
@seisman seisman modified the milestones: 0.8.0, 0.9.0 Dec 11, 2022
@seisman seisman modified the milestones: 0.9.0, 0.10.0 Mar 16, 2023
@seisman seisman modified the milestones: 0.10.0, 0.11.0 Jun 28, 2023
@seisman seisman modified the milestones: 0.11.0, 0.12.0 Dec 3, 2023
@seisman
Copy link
Member Author

seisman commented Dec 19, 2023

We probably want to enable ruff's PTH rules after we finish the replacements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Boring but important stuff for the core devs
Projects
None yet
1 participant