gap-packages / wedderga

Wedderburn Decomposition of Group Algebras

https://gap-packages.github.io/wedderga/

GNU General Public License v2.0

3 stars 6 forks source link

CI: Ensure code coverage reporting will work in February 2022 and beyond #87

Closed fingolfin closed 2 years ago

fingolfin commented 2 years ago

See https://github.com/gap-actions/process-coverage/issues/10

codecov[bot] commented 2 years ago

Codecov Report

Merging #87 (6cf2291) into master (5063de3) will decrease coverage by 0.00%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
- Coverage   83.43%   83.42%   -0.01%     
==========================================
  Files          11       11              
  Lines        5193     5190       -3     
==========================================
- Hits         4333     4330       -3     
  Misses        860      860

Impacted Files	Coverage Δ
lib/crossed.gi	`73.77% <0.00%> (-0.15%)`	:arrow_down:

fingolfin commented 2 years ago

Tests are failing with GAP master due to slightly different output being printed. This isn't caused by this PR, however. To fix it, ideally those tests would be rewritten to not rely on matching specific output so much. Alas, the failing tests come from manual examples... What we just discussed during GAP days is that perhaps manual tests should only be run into a specific set of GAP versions (possibly just one), and perhaps we can provide better tooling to enable just that...