Am 03.12.2020 um 12:46 schrieb Michael Käppler:
[snip]
No, that step remains manually, if I'm not mistaken.
IIUC, a patch that fails 'make check' ('fails' in the sense of 'errors
out') would have failed 'make test' already.
As I understood the (old) process, setting a patch to 'Review' should
not only require a 'passing' 'make check', but also
a visual inspection of the results.
So setting a MR to 'Review' automatically after passing 'make check'
would be a policy change IMHO.
Just to clarify this part: I'm referring to the 'old' system, when
we did run 'make test' in CI.

So

"IIUC, a patch that fails 'make check' ('fails' in the sense of 'errors
out') would have failed 'make test' in the old system, already."

Reply via email to