On Thu, Sep 20, 2018 at 9:43 PM Matthew DeVore <matvore@xxxxxxxxxx> 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 ...