human-essentials icon indicating copy to clipboard operation
human-essentials copied to clipboard

Review "FIXME"s and "TODOs" in our code base

Open cielf opened this issue 2 years ago • 4 comments

Summary

There are a number of "FIXME"'s and "TODOs" in our code base. Review -- each should either get an issue or be deleted.

Details

If you do a search on "FIXME" and "TODO", you'll find them !

Criteria for completion

For each "FIXME" or "TODO", either:

  • [ ] write up the proto-issue, and bring it to the Sunday planning meeting for grooming,
  • [ ] add to a list of "we're probably never going to do this, right"?, to be reviewed or
  • remove the "FIXME" or "TODO"

cielf avatar Apr 23 '23 14:04 cielf

There are a lot of "TODO" comments that are reeeeally low on the "nice to have" list. Do we still want to write up a separate issue for each of them? Maybe one big issue for all of them? Or just agree that they're not that nice to have and be done with it?

dorner avatar May 12 '23 20:05 dorner

Maybe just change the ask above to list them so we can have a review and consensus that we're putting them on the "yeah, we're never gonna get to that" list.

cielf avatar May 27 '23 19:05 cielf

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

github-actions[bot] avatar Jul 25 '23 00:07 github-actions[bot]