You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Was talking through this the other day with the rest of the team. It sounds like smart_todo will try to interpret all TODO(*) comments, and this would conflict with the common "TODO(me):" pattern.
If that's correct, I think this is kind of unreasonable, but I don't have a suggestion for how this could be handled differently.
Anyway, here's a place for discussion about this conflict.
The text was updated successfully, but these errors were encountered:
It sounds like smart_todo will try to interpret all TODO(*) comments
yeah that's correct it will try to parse all comment that starts with TODO(.
However, if the syntax inside the TODO() is not the one expected, it won't do anything.
Does writing TODO(me) creates a problem? It shouldn't
Nope that sounds great then, I just opened this without trying, but it does raise the opposite case... what if the user meant for a todo to be a smart todo but typo’d something? Will it silently not work? Sorry, I’ll just read the code.
Was talking through this the other day with the rest of the team. It sounds like smart_todo will try to interpret all
TODO(*)
comments, and this would conflict with the common "TODO(me):" pattern.If that's correct, I think this is kind of unreasonable, but I don't have a suggestion for how this could be handled differently.
Anyway, here's a place for discussion about this conflict.
The text was updated successfully, but these errors were encountered: