search

bigbio / sdrf-pipelines

A repository to convert SDRF proteomics files into pipelines config files
Apache License 2.0
16 stars 21 forks source link

[MAINT] Favor pathlib over path string manipulation #107

Open fabianegli opened 2 years ago

fabianegli commented 2 years ago

It would be better to not manually work with path strings but use the modern pathlib instead.

https://github.com/bigbio/sdrf-pipelines/blob/837af51f360e81d5b9b6b10a21698fc4c7a3c582/sdrf_pipelines/parse_sdrf.py#L124

fabianegli commented 2 years ago

Note that Path is orange in the above code snippet? That's because it is the name of the Path object in the Python builtin pathlib library and considered an internal variable. If possible these names should not be used for custom variables if not for a very good reason.