MAINT Removes short summary for most builds#21554
Conversation
jjerphan
left a comment
There was a problem hiding this comment.
Trusting you, @thomasjpfan: LGTM.
glemaitre
left a comment
There was a problem hiding this comment.
I am fine skipping the short summary for most of the build and keeping it only for one of them (sometimes it could be useful to check so skipped or xfail)
|
I think it's worth adding that to |
|
Thank you so much for working this out @thomasjpfan. Thinking about it, wouldn't it make sense to directly update the |
That makes sense. I updated the PR to adjust |
ogrisel
left a comment
There was a problem hiding this comment.
LGTM. I wonder if it's really necessary to keep it enabled for pylatest_conda_forge_mkl but this is already a net improvement and I agree we should keep it for at least one of the builds.
Let's merge and we can adjust in a subsequent PR based on experience.
When looking at CI builds, I frequently scroll pass the summary. This PR removes the summary except for one of the builds on the CI.
CC @jjerphan @glemaitre @ogrisel