-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TODO markers should only be detected at comment start #52
Comments
Thanks for reporting this. |
Language is JS. I can do better than provide a sample of my code: I can provide a sample of YOUR code! :) This case is unusal enough to ignore, but the issue could be more of a problem if one used a marker such as FIX, and had a comment such as // This code is to fix a rounding problem |
* Fix incorrectly accept markers in the middle of a string (#52). * Add built packages.
Thanks, will include the fix in the next version 👍 |
Thanks! Let me add that I wouldn't use VS Code if it wasn't for this extension of yours; it's just SO useful. |
Currently, comments such as
// Here, we look for strings containing TODO
are picked up as TODO tasks. This is rarely a problem when using TODO, but can be more of a problem when using other markers. Would it be preferable to ensure that the marker is at the start of the comment text before considering it to be a task?
The text was updated successfully, but these errors were encountered: