Skip to content
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

// Required for regex match #8

Open
evanberkowitz opened this issue Apr 6, 2017 · 1 comment
Open

// Required for regex match #8

evanberkowitz opened this issue Apr 6, 2017 · 1 comment

Comments

@evanberkowitz
Copy link

I couldn't figure out why my notes in python weren't showing up. Looking into lib/messageChecks.js reveals that two slashes are required for a match. But some languages' comments aren't indicated with two slashes---python uses #, for example.

The workaround seems to do, eg. # // TODO: message... but that's very unnatural and easy to forget to do. It would be great if there was a more general way to detect comments (without requiring a lexer for every language, obviously). I tried changing the patterns to (for example) /([\/\/][\/\*]|#)\s*BUG:\s*(?:\(([^:]*)\))*\s*:?\s*(.*)/i but it screws up the message reporting.

@sadiqhabib
Copy link
Contributor

I think you also have problem with OPTIMIZE and FIXME regexp because these are also not working wery well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants