3 ms·
I consider adding TODOs in the code an anti pattern. Those are usually unclear, lack ownership, and purpose of the intended changes gets obsolete. Whenever I re
by 6ue7nNMEEbHcM 5y ago
I consider adding TODOs in the code an anti pattern. Those are usually unclear, lack ownership, and purpose of the intended changes gets obsolete. Whenever I read code with TODOs I find it that more often than not they hurt readability. Usually in reviews I ask to resolve or remove them before merge.
- tobyhinloopen 5y agoWhat if there’s a really tight deadline and you know some code will break soon but will work for now?
- 6ue7nNMEEbHcM 5y agoOpen a bug based on functional (or non functional) impact. Prioritize accordingly. I think if the problem is severe it shouldn't be hidden behind a TODO. If it's not severe then likely it doesn't need attention.