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.

Sign up for GitHub

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

Jump to bottom

Convert to using Pathlib #39

Closed
AdrianDAlessandro opened this issue Aug 29, 2024 · 2 comments · Fixed by #57
Closed

Convert to using Pathlib #39

AdrianDAlessandro opened this issue Aug 29, 2024 · 2 comments · Fixed by #57
Labels
hacktoberfest

Comments

@AdrianDAlessandro
Copy link
Collaborator

All of the path handling is done using strings and os.path this is prone to issues with running on different operating systems. Best practise for path-handling in Python is to use the inbuilt pathlib module.
@tsmbland
Copy link
Contributor

I'll give this one a go

@tsmbland tsmbland mentioned this issue Oct 23, 2024
Merged
@tsmbland
Copy link
Contributor

@AdrianDAlessandro @alexdewar @jmp111 PR created! #57

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use pathlib for path handling tsmbland/Auto-CORPus
3 participants
@alexdewar @tsmbland @AdrianDAlessandro