ghiggi / gpm_api

Global Precipitation Measurement Mission (GPM) python package to download and analyze data with xarray
https://gpm-api.readthedocs.io
MIT License
46 stars 4 forks source link

Enforce `storage` and `transfer_tool` uppercase #52

Closed ghiggi closed 3 months ago

ghiggi commented 3 months ago

Prework

What kind of change does this PR introduce? (check at least one)

Does this PR introduce a breaking change? (check one)

If yes, please describe the impact and communicate accordingly:

The PR fulfills these requirements:

Summary

This PR enforce the storage and transfer_tool acronyms to be upper case in the internal codebase.

Additionally, the availability of the transfer tools (CURL and WGET) is now tested only once at import time !

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 94.54545% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 87.66%. Comparing base (0b87161) to head (9025591).

Files Patch % Lines
gpm/io/download.py 33.33% 2 Missing :warning:
gpm/io/checks.py 95.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #52 +/- ## ======================================= Coverage 87.66% 87.66% ======================================= Files 111 111 Lines 10520 10534 +14 ======================================= + Hits 9222 9235 +13 - Misses 1298 1299 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

coveralls commented 3 months ago

Coverage Status

coverage: 87.669% (+0.007%) from 87.662% when pulling 18545cd7ecd70e2afbdae7902201111fc006dc00 on refactor-storage-acronym into 0b8716139dc5757de214901273615f79bcf66a55 on main.