3 ms·
Usually comes down to one of * I don't see any divide by zero bugs. Approved. * Some inane argument about code style that the linter enforces. * I spent
by worker_person 4y ago
Usually comes down to one of
* I don't see any divide by zero bugs. Approved.
* Some inane argument about code style that the linter enforces.
* I spent 8 hours figuring out what you did, why you did it, reviewing how code matches the requirements, running the code, testing all the scenarios.
* Can we rewrite this entire thing because I prefer a different pattern?
- bil7 4y agohaha, I've definitely seen some of those. Comment standards are worth obeying, personally I like the code review pyramid https://www.morling.dev/blog/the-code-review-pyramid/ https://www.morling.dev/blog/the-code-review-pyramid/