search

JonathanRaiman / epub_conversion

Python package for converting xml and epubs to text files
34 stars 12 forks source link

need to do expanduser #3

Open interrogator opened 4 years ago

interrogator commented 4 years ago

Right now you can't provide a path like ~/Downloads, because ~ is not expanded...

JonathanRaiman commented 4 years ago

@interrogator which method should be receiving this path/where do you expect this to work?

Would be happy to look into this -- I've generally seen expanduser done in top-level/CLI places because you expect user input, but here I'm not sure there's a place where it would go

interrogator commented 4 years ago

Right after you accept an input path by Converter(path), the path should be made as 'normalised' as possible before you store it as self.path