Closed ghiggi closed 4 months ago
Attention: Patch coverage is 85.65121%
with 65 lines
in your changes are missing coverage. Please review.
Project coverage is 81.75%. Comparing base (
8bd40e5
) to head (d611a32
). Report is 14 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
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:
This PR introduce several BIG breaking changes.
It modify the way on how users imports the software. From:
import gpm_api
toimport gpm
It modify the name of the gpm_api xarray accessor. Example: From
ds.gpm_api.plot_map()
tods.gpm.plot_map()
It modify the name of the config file. From
config_gpm_api.yml
toconfig_gpm.yaml
The PR fulfills these requirements:
[x] It's submitted to the branch named as follow :
bugfix-<some_key>-<word>
doc-<some_key>-<word>
tutorial-<some_key>-<word>
feature-<some_key>-<word>
refactor-<some_key>-<word>
optimize-<some_key>-<word>
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g.
fix #xxx[,#xxx]
, where "xxx" is the issue number)[ ] Don't forget to link PR to issue if you are solving one.
[ ] All tests are passing.
[ ] New/updated tests are included