search

pradyunsg / furo

A clean customizable documentation theme for Sphinx
https://pradyunsg.me/furo/quickstart
MIT License
2.65k stars 304 forks source link

src/furo/init: do not use os.path #632

Closed umarcor closed 1 year ago

umarcor commented 1 year ago

This PR removes the single use of os. in src/furo/__init__.py, in favour of Path.

pradyunsg commented 1 year ago

I don't think that this inherently has any advantages, and thus am gonna let this stay as-is. Thanks for filing this PR none the less. :)

umarcor commented 1 year ago

@pradyunsg it's a cleanup for consistency, since the rest of the document uses pathlib already. So, it is expected to be functionally equivalent, not introducing any change (neither advantage nor disadvantage).