BAMWelDX / weldx

The welding data exchange format
https://www.bam.de/weldx
BSD 3-Clause "New" or "Revised" License
19 stars 9 forks source link

Update mypy action #847

Closed CagtayFabry closed 1 year ago

CagtayFabry commented 1 year ago

Changes

Update mypy GitHub action:

see #845

Checks

CagtayFabry commented 1 year ago

Is this what you had in mind with #845 @marscher ?

The local pip install doesn't seem to make a difference here

github-actions[bot] commented 1 year ago

Test Results

2 187 tests  ±0   2 186 :heavy_check_mark: ±0   2m 41s :stopwatch: +11s        1 suites ±0          1 :zzz: ±0         1 files   ±0          0 :x: ±0 

Results for commit 863015bb. ± Comparison against base commit ac3eda4a.

:recycle: This comment has been updated with latest results.

codecov[bot] commented 1 year ago

Codecov Report

Merging #847 (863015b) into master (ac3eda4) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #847   +/-   ##
=======================================
  Coverage   96.60%   96.60%           
=======================================
  Files          95       95           
  Lines        6222     6222           
=======================================
  Hits         6011     6011           
  Misses        211      211           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

marscher commented 1 year ago

Is this what you had in mind with #845 @marscher ?

yes, it is. It is kinda strange, that with the same mypy version, configuration, and stub packages I get very different results.