News for hackers. Stuff that matters.

A comment on TODOs Aren't for Doing. Parent comment

Re:TODOs Aren't for Doing by OrderlyTiamat on Tuesday July 22, 2025 @04:01PM (#44652320)
You can leave the TODO in the comments- e.g. ruff the linter has an optional rule to disallow TODO comments unless it's followed by an issue url.

If you put that in the CI, then you can use TODOs either as blockers you wish to fix before merging, or as long term comments to be fixed in a future ticket.

[ Parent | on HN ]
Re:TODOs Aren't for Doing by evnu on Wednesday July 23, 2025 @01:45AM (#44656038)
Some years ago, I started to use FIXME to indicate that something is blocking the PR and needs to be done before merging, and TODO if something can be done at a later point in time. Then, CI only needs to grep for FIXME to block merging the PR, which works for practically any language. Works pretty well for me, maybe that tip can help others as well.
[ Parent | on HN ]