▲ | com2kid 18 hours ago | |
> Come back in a week and update us on how long you've spent debugging all the ways that the code was broken that you didn't notice in those 15 minutes. Same as if I let a junior engineer merge code to main w/o unit tests. Complete garbage, of course. Oh wait, my code is also trash w/o good unit tests, because I am only human. Instead I'll write out a spec, define behaviors and edge cases, and ask the junior engineer to think about them first. Break implementation down into a plan, and I'll code review each task as it is completed. Now all of a sudden, the code is good, independent of who/what wrote it! | ||
▲ | scubbo 2 hours ago | parent [-] | |
Yep, precisely my point. Reviewing a one-shot in 15 minutes is irresponsible. |