<{^_^}>
[ofborg] @cole-h opened pull request #458 → {checkout,clone}: redirect output to /dev/null → https://git.io/JvxxJ
cole-h has quit [Quit: Goodbye]
cole-h has joined #nixos-borg
<cole-h>
gchristensen: in ~3 hours I'll explain each case where redirecting stderr silences git reporting "as normal but not an error" (lecture in the meantime) and we can go from there :)
<cole-h>
I ran all of the commands I changed with only stdout pointing to `Stdio::null()` and recorded the results in comments below the function
<gchristensen>
cool
<gchristensen>
let's keep stderr output for all of them, for now
<cole-h>
Roger.
<cole-h>
Can I remove stderr for the `git am --abort` stuff, considering they should always fail (and print to stderr)? Or keep it as well
<cole-h>
`git am --abort` and `git merge --abort`, specifically
<gchristensen>
sure
<cole-h>
(also, don't merge immediately; I'm going to sneak a commit that changes the `issue.id` to `issue.number` so we can actually see what PR is being operated on, instead of numbers in the 100s of millions)
<cole-h>
(please)
<gchristensen>
cool :D
<cole-h>
I'll push after you re-review, in case I need to make more changes (save CI from itself) ;^)
<gchristensen>
lgtm cole-h
<cole-h>
Cool with stdout + stderr -> /dev/null on merge and am aborts?