|
|
|
|
|
by jmiserez
3219 days ago
|
|
You can’t catch all the issues in code review every time, and it’s not always you that does the reviews. If you fix everything yourself, the original committer won’t learn anything and the next submissions will have the same issues. Or perhaps the edge case isn’t really an edge case and there is nothing to fix after all, in which case you’ve wasted time for nothing. Also, if you do this people will likely subconciously take advantage [1] of it in the long run, and if you ever leave the team quality will suffer. [1] ever had flatmates? Do all the dishes and cleaning for a month and you’ll see :-/ |
|