search

pytroll / pytroll-collectors

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

Remove 'six' as requirement #88

Closed pnuu closed 3 years ago

pnuu commented 3 years ago

This PR removes the now-obsolete usage of six compatibility library, closes #87 .

There were also two "hacks" to make Python 2 work, which were removed. And the headers in each file were harmonized.

codecov[bot] commented 3 years ago

Codecov Report

Merging #88 (ff51e87) into main (71d66a1) will increase coverage by 0.06%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #88      +/-   ##
==========================================
+ Coverage   83.26%   83.33%   +0.06%     
==========================================
  Files          19       19              
  Lines        2624     2622       -2     
==========================================
  Hits         2185     2185              
+ Misses        439      437       -2
Impacted Files Coverage Δ
pytroll_collectors/__init__.py 100.00% <ø> (ø)
pytroll_collectors/file_notifiers.py 0.00% <ø> (ø)
pytroll_collectors/tests/test_helper_functions.py 87.50% <ø> (ø)
pytroll_collectors/tests/test_region_collector.py 92.85% <ø> (+0.12%) :arrow_up:
pytroll_collectors/tests/test_scisys.py 94.79% <ø> (ø)
pytroll_collectors/tests/test_segments.py 99.19% <ø> (ø)
pytroll_collectors/triggers/__init__.py 70.00% <ø> (ø)
pytroll_collectors/triggers/_inotify.py 34.21% <ø> (ø)
pytroll_collectors/triggers/_posttroll.py 90.56% <ø> (ø)
pytroll_collectors/triggers/_watchdog.py 39.65% <ø> (ø)
... and 5 more

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 71d66a1...ff51e87. Read the comment docs.