ramp-workflow icon indicating copy to clipboard operation
ramp-workflow copied to clipboard

MAINT: replace os.path with pathlib.Path

Open rth opened this issue 1 year ago • 1 comments

This package does a lot of path manipulations, and it would be nice to use pathlib.Path more. See https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/ for motivation of using pathlib.

This is particularly useful in combination with typing #316 since path then would be of type Path instead of str. The code would also be much sorter (less of os.path.join everywhere) and more readable.

A potential maintenance task cc @SofianChay

rth avatar Sep 14 '22 13:09 rth

I second it.

kegl avatar Sep 16 '22 16:09 kegl