Thread (2 messages) flat view 2 messages, 2 authors, 2021-07-16

Re: [PATCH] ci: run `make sparse` as a GitHub workflow

From: Junio C Hamano <hidden>
Date: 2021-07-14 22:27:37

Johannes Schindelin [off-list ref] writes:
quoted
It wasn't quite obvious why we justify spending 370 minutes one more
time only to rerun 30-second job, though.
True.

And this is not a new problem. Every time anything happens in those
`osx-gcc` or `osx-clang` jobs (e.g. that transient problem with the broken
pipes in t5516 [*1*], that's a fun one), a full re-run is necessary, or
else the commit and/or PR will remain marked as broken.

In other words, while it is totally appropriate for me to explain this to
you in this here thread because it came up as a tangent, it would be
inappropriate to stick that explanation into this patch's commit message.
We do not make a habit of adding tangents that came up during patch
reviews into commit messages, and I do not intend to start such a habit
here, either.
I do not agree; a brief mention "even though piling more and more on
the primary workflow would make it even less convenient to re-run,
it is already so bad that another one would not hurt too much more"
would be a clue good enough to motivate others to do something about
it if they feel like it.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help