12 ms·
> make sure the comments are updated while you're in the code. You can also look at them in a code review. People will invariably forget to update comments and
by u801e 5y ago
> make sure the comments are updated while you're in the code. You can also look at them in a code review.
People will invariably forget to update comments and unless the comments show up in context lines of the diff associated with a change, it's likely that reviewers will overlook the need to change them.
- Gibbon1 5y agoYou comment is a standard explanation for not having comments. Always makes me wonder why stale comments aren't flagged during code review? Seriously failure to update comments should eventually lead to constructive dismissal.
- sokoloff 5y agoI don't think "constructive dismissal" is the phrase is you want there. (That's normally associated with employer wrongdoing or, at a minimum, mischaracterizing the terms of an employee's departure..)