aimclub / GOLEM

Graph Optimiser for Learning and Evolution of Models
https://thegolem.readthedocs.io
BSD 3-Clause "New" or "Revised" License
63 stars 7 forks source link

Hotfix condition on telegram notification of integration tests #132

Closed MorrisNein closed 1 year ago

MorrisNein commented 1 year ago

One part of the complex condition wasn't tested, since the scheduled workflow runs on the main branch only.

After merge to main, the full condition always returns true.

Most likely, the condition is interpreted as a literal, not as an expression.

codecov-commenter commented 1 year ago

Codecov Report

Merging #132 (e533576) into main (522878e) will decrease coverage by 0.07%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
- Coverage   72.38%   72.31%   -0.07%     
==========================================
  Files         128      128              
  Lines        7336     7336              
==========================================
- Hits         5310     5305       -5     
- Misses       2026     2031       +5     

see 4 files with indirect coverage changes