|
|
|
|
|
by pornel
328 days ago
|
|
Tracking in external system adds overhead not only for filing the issue, but also for triaging it, backlog management, re-triaging to see if it's still a problem, and then closing it when it's finished. Issues in an external systems may also be overlooked by developers working on this particular code. There are plenty of small things that are worth fixing, but not worth as much as the overhead of tracking them. TODO in code is easy to spot when someone is working on this code, and easy to delete when the code is refactored. |
|
In my experience, these can often be replaced with logger.Error("the todo") or throw new Exception("the todo"), which read about as well as //TODO: the todo, but also can bubble up to people not actually reading the code. Sometimes, though, there's no simple test to trigger that line of code, and it just needs to be a comment.