PyPSA / linopy

Linear optimization with N-D labeled arrays in Python
https://linopy.readthedocs.io
MIT License
153 stars 40 forks source link

ci: fix potential setuptools_scm issues in gh actions #298

Closed lkstrp closed 3 weeks ago

lkstrp commented 1 month ago

In reference to https://github.com/PyPSA/PyPSA/pull/907 #279 and #292 I just add a fetch for the tags here so that setuptools_scm can detect the version correctly. Right now that doesn't lead to errors, so only to avoid the potential bug search in the future (which I just had).

Fore more details see here.

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 86.72%. Comparing base (07aa9f6) to head (d43bef3).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #298 +/- ## ========================================== - Coverage 89.72% 86.72% -3.00% ========================================== Files 16 16 Lines 4037 4037 Branches 942 942 ========================================== - Hits 3622 3501 -121 - Misses 282 412 +130 + Partials 133 124 -9 ```

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

FabianHofmann commented 1 month ago

flake 8 does not like some syntax here https://github.com/PyPSA/linopy/actions/runs/9273219169/job/25512792210?pr=298 (?)

FabianHofmann commented 3 weeks ago

don't know what is going on with codecov, I am closing and opening the pr in the hope that codecov with work then