CI: make codecov checks non-failing

[FliegendeWurst]

Insufficient test coverage shouldn't lead to a red X in the PR list. The ❌ should be reserved for more urgent problems (formatting / compilation / failing tests) that actually block merging.

Docs for codecov.yml: https://docs.codecov.com/docs/commit-status

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0ab2555) 37.86% compared to head (0de9765) 37.86%.
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##               main    #3346   +/-   ##
=========================================
  Coverage     37.86%   37.86%           
+ Complexity    16904    16903    -1     
=========================================
  Files          2055     2055           
  Lines        125528   125528           
  Branches      21226    21226           
=========================================
+ Hits          47525    47526    +1     
  Misses        72152    72152           
+ Partials       5851     5850    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[unp1]

Is it possible to just raise the percentage bar from which on a red cross is used. Setting it to 0.5 % or 1% would get rid of all the usual ones, but still highlight more severe losses of coverage?

[Drodt]

Is it possible to just raise the percentage bar from which on a red cross is used. Setting it to 0.5 % or 1% would get rid of all the usual ones, but still highlight more severe losses of coverage?

There is a threshold option, which should work for our case: https://docs.codecov.com/docs/github-3-customizing-codecov

[FliegendeWurst]

There is a threshold option, which should work for our case: docs.codecov.com/docs/github-3-customizing-codecov

Good idea. I think target: 38% and threshold: 1% would work well. When the test coverage increases significantly, the target should be increased as well.

[FliegendeWurst]

Hold up, I didn't change it to 38% +- 1 yet. But that can be done in a follow-up PR if it's desired ..