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 pathlib #20

Closed
jodal opened this issue Dec 9, 2019 · 1 comment · Fixed by #30
Closed

Convert to pathlib #20

jodal opened this issue Dec 9, 2019 · 1 comment · Fixed by #30
Assignees
@jodal
Labels
C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal
Milestone
v3.1.0

Comments

@jodal
Copy link
Member

jodal commented Dec 9, 2019

To ensure proper handling of file systems with unknown encodings or even arbitrary bytes in the file names, we should fully convert Mopidy-Local to use pathlib instead of os.path.
@jodal jodal added this to the v3.0.0 milestone Dec 9, 2019
@jodal jodal changed the title Fully convert to pathlib Convert to pathlib Dec 9, 2019
@jodal jodal added the C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal label Dec 21, 2019
@jodal jodal removed this from the v3.0 milestone Dec 21, 2019
@jodal
Copy link
Member Author

jodal commented Jan 2, 2020

This is mostly fixed by #29, with only a tiny bit left in storage.py.

@jodal jodal added this to the v3.1.0 milestone Jan 2, 2020
@jodal jodal mentioned this issue Jan 3, 2020
Merged
@jodal jodal closed this as completed in #30 Jan 4, 2020
@jodal jodal self-assigned this Jan 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jodal jodal

Labels
C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal
Projects
None yet
Milestone
v3.1.0
Development

Successfully merging a pull request may close this issue.

Replace os.path in storage module
1 participant
@jodal