HERA-Team / hera_qm

HERA Data Quality Metrics
MIT License
2 stars 2 forks source link

ci: add warnings tests and fix some warnings #454

Closed steven-murray closed 6 months ago

steven-murray commented 6 months ago

This adds a pytest run with warnings that error, so we can catch them sooner rather than later.

I went through and fixed some of the errors that popped up. Some other warnings are fixed in #453, so they will disappear once that is merged. Other warnings I have silenced if they seem reasonable to let go, but I would advise quickly looking over all the "filterwarnings" that I added to make sure that you agree that I'm not hiding anything important.

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (cb86b39) 97.06% compared to head (695e64e) 97.07%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #454 +/- ## ======================================= Coverage 97.06% 97.07% ======================================= Files 11 11 Lines 3549 3553 +4 ======================================= + Hits 3445 3449 +4 Misses 104 104 ```

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