search

earthlab / matplotcheck

A python package for checking and testing matplotlib plots. We use this for autograding student assignments but there are many other potential use cases including package testing (for packages with plots)!
https://matplotcheck.readthedocs.io
BSD 3-Clause "New" or "Revised" License
18 stars 8 forks source link

Move from Travis / Appeveyor to GitHub actions (#369) #372

Closed lwasser closed 3 years ago

lwasser commented 3 years ago

update main branch

codecov[bot] commented 3 years ago

Codecov Report

Merging #372 (78fb13f) into main (ce9a006) will increase coverage by 0.67%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #372      +/-   ##
==========================================
+ Coverage   81.92%   82.59%   +0.67%     
==========================================
  Files          21       22       +1     
  Lines        2030     1954      -76     
==========================================
- Hits         1663     1614      -49     
+ Misses        367      340      -27
Flag Coverage Δ
unittests 82.59% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
matplotcheck/tests/conftest.py 78.94% <0.00%> (-3.08%) :arrow_down:
matplotcheck/tests/test_points.py 92.30% <0.00%> (-0.47%) :arrow_down:
matplotcheck/tests/test_polygons.py 91.66% <0.00%> (-0.40%) :arrow_down:
matplotcheck/tests/test_lines.py 93.67% <0.00%> (-0.38%) :arrow_down:
matplotcheck/cases.py 0.00% <0.00%> (ø)
matplotcheck/tests/test_raster.py 100.00% <0.00%> (ø)
matplotcheck/tests/test_base_data.py 100.00% <0.00%> (ø)
matplotcheck/tests/test_base_lines.py 100.00% <0.00%> (ø)
matplotcheck/tests/test_timeseries_module.py 100.00% <0.00%> (ø)
setup.py 0.00% <0.00%> (ø)

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 ce9a006...78fb13f. Read the comment docs.

lwasser commented 3 years ago

also @nkorinek geopandas is not now going to require descartes!! we just have to figure out what version is released with it removed as a dep :) !!!

nkorinek commented 3 years ago

@lwasser looks like the last release they did was in june of this year, so they're probably due for one soon. The descartes change was added on the 20th of december, so we probably have a bit of time before it's added to a release. Aside from this this pr looks good though!