Hacker News new | ask | show | jobs
by redcap 3323 days ago
#1 There's nothing wrong with TODO comments. Often you don't want implement feature B in full so that you can focus on feature A. Of course you've told your customers that you'll do both features, so you push feature B back to a future release. No problem in using TODO there - at least you have comments as to what's actually going on and what may need to be fleshed out in the future.
1 comments

Normally the argument is the this should be part of your backlog and not spread all over the code where no one (and especially the PO) can keep track of all the TODOs in all the files.
Depends on trust. I've had an experience where old tickets documenting important architectural problems (e.g. SQL injection) were mass-closed as "probably obsolete" by managerial types.

I was very happy I left a TODO in the actual code calling out the problem, because I could find and reopen the wrongly-closed ticket.

Heh. Visual Studio actually does keep track of all the TODOs in all the files in your project. It's the Task List window (View -> Other Windows -> Task List, or ctrl-alt-K).
Does it track FIXMEs?
By default it looks for the tokens: HACK, TODO, UNDONE, and NOTE.

However, you can add your own tokens and assign different priorities to them.

https://docs.microsoft.com/en-us/visualstudio/ide/using-the-...

Very nice, it now almost resembles the functionality of "grep -rn FIXME ." in the shell and ":grep -r FIXME ." in vim.