search

pytroll / pytroll-collectors

Collector modules for Pytroll
GNU General Public License v3.0
3 stars 18 forks source link

Confirm configuration file was read in gatherer. #109

Closed gerritholl closed 2 years ago

gerritholl commented 2 years ago

Confirm the configuration file was read successfully when initialising the geographic gatherer.

Fixes #72.

codecov[bot] commented 2 years ago

Codecov Report

Merging #109 (eb5810b) into main (9225601) will increase coverage by 0.03%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #109      +/-   ##
==========================================
+ Coverage   89.58%   89.61%   +0.03%     
==========================================
  Files          25       25              
  Lines        3609     3620      +11     
==========================================
+ Hits         3233     3244      +11     
  Misses        376      376
Impacted Files Coverage Δ
pytroll_collectors/geographic_gatherer.py 85.71% <100.00%> (+0.21%) :arrow_up:
...troll_collectors/tests/test_geographic_gatherer.py 98.13% <100.00%> (+0.08%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 9225601...eb5810b. Read the comment docs.

gerritholl commented 2 years ago

I'm not authorised to request reviews here, but maybe @mraspaud @TAlonglong @pnuu can try this out in their workflow? It does have the potential to break compatibility in theory, but I don't think it should in practice.