On Thu, Sep 20, 2018 at 9:43 PM Matthew DeVore <matv...@google.com> wrote: > Add two guidelines:
Probably s/two/three/ or s/two/several/ since the patch now adds three
guidelines.
> - pipe characters should appear at the end of lines, and not cause
> indentation
The "not cause indentation" bit is outdated since the added guideline
no longer says this.
> - pipes should be avoided when they swallow exit codes that can
> potentially fail
And:
- $(git ...) should be avoided ...
