Thursday, September 17, 2009 11:13 AM Sean Feldman

TODOs are not forgotten

We are using Hudson as a build server, and one of the lasts steps that were taken is to mark a build as ‘unstable’ when we pass a certain number of TODO comments in our code (an arbitrary number). While I am not a 100% sold on a number, I think it’s a good way of insuring things are not just marked and forgotten. Actually, we are not even tracking TODOs, but BROKEN_WINDOW comments, as those are definitely bad. Failing on HACK is another possibility. Visualization plays a significant role in my case (interpretation of things based on visualization), and here how it looks (green is all good, yellow is all passed, but number of comments has exceeded the limit).

image

Filed under:

Comments

# re: TODOs are not forgotten

Thursday, September 17, 2009 1:18 PM by CCNA Training

Thanks for the sharing!

# re: TODOs are not forgotten

Thursday, September 17, 2009 2:48 PM by Brian

I like this idea. How about having it check the number of not implemented exceptions that get thrown as well? That could be just as important as a TODO imo.

# re: TODOs are not forgotten

Thursday, September 17, 2009 5:43 PM by Sean Feldman

@Brian,

I am sure you can leverage that with Hudson as well :)