3 ms·
the old/existing code was all underscore, they wanted to use camelcase instead. it's a dumb thing to be argue about I know but it made the code review harder wh
by ge96 2y ago
the old/existing code was all underscore, they wanted to use camelcase instead. it's a dumb thing to be argue about I know but it made the code review harder when instead of 10s of line diffs there's almost a hundred granted easy to see just changing casing
- willseth 2y agoI just insist that style only changes go in a separate commit.
- JTyQZSnP3cQGa8B 2y agoAnd when it impacts a lot of files, it can break the compiler or introduce bugs. It MUST go in its own PR/MR.