3 ms·

At Google, the team that I worked on required the format `// TODO(name): [Tracking Bug#] comment ` In other places, I've seen just blanket `// TODO: comment` i

by Johnie 10y ago

At Google, the team that I worked on required the format `// TODO(name): [Tracking Bug#] comment ` In other places, I've seen just blanket `// TODO: comment` is pretty bad. There's no accountability and also you don't know when it's obsolete.

hinkley 10y ago
Sometimes the TODO message doesn't make enough sense, and having the author gives you who to ask for context.
- koverstreet 10y ago
  That's what git blame is for (I've got a thing in vim that runs git blame on the current line, and pops up the commit message in the quickfix window).
  - hinkley 10y ago
    Git blame only works until code starts shifting around.