3
devphobe
13d

`if (boolean)`

Would you tell a colleague that their PR needed to be refactored because they used parenthesis around an if block that didn't need it? I'm all about explicit order of operations and often add brackets and parentheses around things that don't need them. It's cool if you don't want to put them in, but does the PR need to be held up over this?

Comments
  • 2
    Depends on the guidelines.
    If there are early returns allowed I would like to see the brackets at all times, otherwise I don't care
Add Comment