Enhance code coverage bot message.

[codemzs]

No description provided.

[sharwell]

I've used this form in the past and it causes so much noise it's just distracting. It actually used to be my preferred view, but once I switch to the simpler view the improvement was obvious. I recommend not making this change.

[codemzs]

@sharwell Thanks for your input. Some of us actually like this format, I will take a vote on this and decide.

CC: @ViktorHofer

[codecov]

Codecov Report

Merging #2235 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@ Coverage Diff @@ ## master #2235 +/- ## ========================================== - Coverage 71.54% 71.54% -0.01%  ========================================== Files 800 800 Lines 141847 141847 Branches 16119 16119 ========================================== - Hits 101480 101478 -2  - Misses 35916 35917 +1  - Partials 4451 4452 +1

Flag	Coverage Δ
#Debug	71.54% <ø> (-0.01%)	⬇️
#production	67.83% <ø> (-0.01%)	⬇️
#test	85.73% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
...StandardLearners/Standard/LinearModelParameters.cs	60.63% <0%> (-0.27%)	⬇️
...soft.ML.Data/DataLoadSave/Text/TextLoaderCursor.cs	82.66% <0%> (-0.21%)	⬇️

[rogancarr]

Approve the change, but don't have strong feelings about the view. 🎈

[ViktorHofer]

You don't need the behavior: default.

[justinormont]

Would it be possible to put the extra info in a collapsible box?

CLICK ME

Impacted Files	Coverage Δ
...soft.ML.TestFramework/DataPipe/TestDataPipeBase.cs	70.05% <0%> (-0.33%)	⬇️
...soft.ML.Data/DataLoadSave/Text/TextLoaderCursor.cs	82.86% <0%> (+0.2%)	⬆️
...StandardLearners/Standard/LinearModelParameters.cs	60.9% <0%> (+0.26%)	⬆️
src/Microsoft.ML.Maml/MAML.cs	26.21% <0%> (+1.45%)	⬆️

via: https://gist.github.com/joyrexus/16041f2426450e73f5df9391f7f7ae5f

[sharwell]

@justinormont It is not currently possible. I requested that behavior from codecov a long time back and will send them a reminder.

[TomFinley]

Am I the only one that can't see any information out of codecov at all? I have a spinning webpage that's been "loading" for minutes. (Then fails with a "codecov.io unexpectedly closed the connection".)

Unrelated: it seems like the codecov build is actually blocking the gate? The entire purpose of adding it as a separate build was so it would not do that.

[codemzs]

@TomFinley link to the PR please? CodeCoverge-CI should not block the merge since it’s not the required step?

[codemzs]

@TomFinley Below is the snapshot of the scenario where main CI has finished but code coverage CI has not and you can still merge:

[codemzs]

@sharwell The PR view isn't working so it will be good to have a snapshot of what files are affected even though it won't show the whole list but it will list the top files.