econchick / interrogate

Explain yourself! Interrogate a codebase for docstring coverage.
https://interrogate.readthedocs.io
MIT License
562 stars 46 forks source link

Replaced `os.path` with `pathlib` #182

Open trag1c opened 1 month ago

trag1c commented 1 month ago

Hey, I just made a Pull Request!

Description

Changes all occurrences of os.path to use pathlib instead.

Motivation and Context

Closes #179

Have you tested this? If so, how?

Updated unit tests :)

Checklist for PR author(s)

Release note

NONE
trag1c commented 1 month ago

Not sure if this deserves a release note or any versionchanged directives.

I'll take a look at Windows' fails tomorrow :+1:

trag1c commented 1 month ago

I'll take a look at Windows' fails tomorrow 👍

Took a bit longer but I'm getting OSErrors, seems like Windows can't install cairosvg 🤔 I feel like I'm doing something wrong here.

@econchick Could you approve the workflow? 😄