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
Comments with words partially containing the word "todo" are marked the todo-syntax error, even if it is not a TODO comment.
E.g. in a comment with the word Todorov (could be a name of a method, class, variable, etc.), the issue happens, and I have to either disable tailor on the comment or misspell the word. Which are both bad, since I want to keep tailor on, while I correctly spell the word in the comment.
I suggest assuming the todo-syntax mark only if there is a TODO word that, either:
is NOT substring of another (like in "Todorov")
is the first word of the comment
is all in capital case.
The text was updated successfully, but these errors were encountered:
Thanks for reporting @joaoscalett! Based on the original intent of this rule, we've decided to switch to only flagging comments that contain "todo" as isolated words when they are in an incorrect format (as you've suggested). However, we will still flag instances regardless of case and location of the word because we want to encourage the "preferred syntax" when TODO tasks are mentioned later in a comment or with different capitalization.
Thanks for your suggestions, the fix is forthcoming!
Comments with words partially containing the word "todo" are marked the todo-syntax error, even if it is not a TODO comment.
E.g. in a comment with the word Todorov (could be a name of a method, class, variable, etc.), the issue happens, and I have to either disable tailor on the comment or misspell the word. Which are both bad, since I want to keep tailor on, while I correctly spell the word in the comment.
I suggest assuming the todo-syntax mark only if there is a TODO word that, either:
The text was updated successfully, but these errors were encountered: