DEV Community

Discussion on: Performing a PR Review

Collapse
 
arne_mertz profile image
Arne Mertz

I fully agree with this. In the grand scheme, indentation and brace styles do not matter much and, in the end, we will be able to understand the code (or not) regardless of which style was chosen.
But that's exactly why anyone submitting a PR should not be too lazy to stick to the existing style. Having different styles mixed together is just an unnecessary annoyance and distraction for any reader. Consistency is key.

Collapse
 
dijitalmunky profile image
Chris Roe

Well said!