4 ms·
Sounds to me that the real takeaway is that the essentials you list don't, in fact, matter if you have incompetent developers.
by brazzy 4y ago
Sounds to me that the real takeaway is that the essentials you list don't, in fact, matter if you have incompetent developers.
- RugnirViking 4y agowell certainly if you have incompetent developers in senior management, but is that a surprise to anyone? I don't think a good git workflow or whatever could ever fix something like that
- gorjusborg 4y agoTo me, code review is a tool that can make incompetence a temporary condition. People have to be open to constructive feedback and want to grow, however.
- jiggawatts 4y agoHaving a code review policy is like having speed limit signs on the motorway. Without enforcement, you can make the argument that speed limits don’t save lives because everyone drives as fast as they can anyway. Same thing for code review. If it’s not policed by management, then it does nothing. I saw one place that would send alert emails if a commit was approved too fast, assuming that someone must have had it insta-approved by a friend as a favour.