This adapter is only ever run by the Radicle CI broker, and that can
now limit CI run duration itself. Thus having this adapter do it too
is pointless duplication.
Further, this removes the run time dependency on an external "timeout"
command, which turns out not to be portable: it isn't available on
Macs, by default.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
This makes it easier, for me, to follow the output. It's more like
what I see in the terminal if I run the commands directly. If this
turns out to be bad for others, I'll revert.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
Previously, every run log would contain:
Note: switching to 'HEAD^'.
You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.
If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:
git switch -c <new-branch-name>
Or undo this operation with:
git switch -
Turn off this advice by setting config variable advice.detachedHead to false
HEAD is now at 2d0d98e feat: add branch or patch information to run log
This should now be gone.
Signed-off-by: Lars Wirzenius <liw@liw.fi>
If any command run by CI for a repository fails, set the run to failed
in the run info builder.
Also, we were hiding that failure with later errors, if, say, updating
a log file or removing temporary files failed, so don't do that.
In the old engine it's hard to follow how various errors are handled.
This new engine should make it clearer.
This isn't entirely a pure refactoring. It subtly changed how the
external behavior is, to fix problems.
Signed-off-by: Lars Wirzenius <liw@liw.fi>