2 ms·
The way I've seen other people try to combat it is by taking the "commit early, commit often" thing to an extreme with small commits. My manager likes sub-100
by curryst 6y ago
The way I've seen other people try to combat it is by taking the "commit early, commit often" thing to an extreme with small commits. My manager likes sub-100 line MRs, which gets restrictive in more verbose languages like Go. It would be hard to find more than 3 flaws that are serious enough to need review in that little space.
The tradeoff is that now you're constantly reviewing code where you have no idea how it fits into the rest of the design, which I find a lot less useful. It's good for catching things that a linter probably should have caught anyways, but it's really bad for critiquing how concurrency is done or other higher level constructs.