Closed danielfromearth closed 4 months ago
Github Issue: #114
This adds type annotations throughout podaac.merger, especially to clarify the usage of pathlib.Path instead of str for file paths. It's not complete in adding all annotations, but brings up the level of annotations quite a bit.
podaac.merger
pathlib.Path
str
n/a
See Pull Request Review Checklist for pointers on reviewing this pull request
Hi @danielfromearth Is this ready to go? Can you add us as reviewers? looks great tho btw!
Github Issue: #114
Description
This adds type annotations throughout
podaac.merger
, especially to clarify the usage ofpathlib.Path
instead ofstr
for file paths. It's not complete in adding all annotations, but brings up the level of annotations quite a bit.Overview of verification done
n/a
Overview of integration done
n/a
PR checklist:
See Pull Request Review Checklist for pointers on reviewing this pull request