interrogate icon indicating copy to clipboard operation
interrogate copied to clipboard

Replace `os.path` with `pathlib`

Open trag1c opened this issue 1 year ago • 3 comments

pathlib offers a simpler and more human-friendly way of dealing with paths and it seems like interrogate mostly uses os.path.

I believe changing this would be more in line with CONTRIBUTING.md :wink:

interrogate tries to take advantage of modern features whenever possible

I'd be happy to work on this if you find this issue reasonable :)

trag1c avatar Apr 20 '24 21:04 trag1c

Hey @trag1c - certainly would welcome a patch for this. I can't remember why os.path was originally chosen, it may have been something to do with Windows machines. Or I might be making it up. Either way, so long as it works on Windows + Unix, I'd appreciate it!

econchick avatar Apr 23 '24 15:04 econchick

Roger that! Though I think I'll wait for #178 (where I've left a few tips ^^) to merge so that it's easier for me to make the transition if that's okay 😄

trag1c avatar Apr 23 '24 16:04 trag1c

#178 is all set (which you've probably seen) - sorry for the lag!

econchick avatar May 20 '24 15:05 econchick