Closed ghiggi closed 3 months ago
Attention: Patch coverage is 88.31325%
with 97 lines
in your changes are missing coverage. Please review.
Project coverage is 87.86%. Comparing base (
03a7204
) to head (21548d4
).
Files | Patch % | Lines |
---|---|---|
gpm/visualization/profile.py | 0.00% | 15 Missing :warning: |
gpm/dataset/crs.py | 75.92% | 13 Missing :warning: |
gpm/dataset/conventions.py | 42.85% | 8 Missing :warning: |
gpm/utils/checks.py | 85.36% | 6 Missing :warning: |
gpm/io/local.py | 64.28% | 5 Missing :warning: |
gpm/retrievals/routines.py | 0.00% | 5 Missing :warning: |
gpm/tests/utils/hdf5.py | 0.00% | 5 Missing :warning: |
gpm/accessor/methods.py | 76.47% | 4 Missing :warning: |
gpm/dataset/dataset.py | 0.00% | 4 Missing :warning: |
gpm/dataset/datatree.py | 20.00% | 4 Missing :warning: |
... and 16 more |
: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:
The PR fulfills these requirements:
bugfix-<some_key>-<word>
doc-<some_key>-<word>
tutorial-<some_key>-<word>
feature-<some_key>-<word>
refactor-<some_key>-<word>
optimize-<some_key>-<word>
fix #xxx[,#xxx]
, where "xxx" is the issue number)If adding a new feature, the PR's description includes:
Summary
Add several ruff pre-commit rules and fix code to pass the pre-commit checks.