-
Notifications
You must be signed in to change notification settings - Fork 20
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
Add TODO to an avoidance list as a warning #78
Comments
Let's call that an ignore list or something. ;) |
Actually, we should avoid using |
I think this is a good idea, with a caveat or two. I think if you have a "TODO" in your documentation it should be a known issue which requires work to be done. If a Jira issue has been created for the work to be done, then I think the TODO should be ignored automatically. So some specific implementation of This would mean any usage of "TODO" would either have a linked issue which requires work or raise an error - which would mean it could be easily noticed or tracked. Thinking about general implementation of this rule (as I hope that we'll be able to provide our Vale style as a Vale package at some point) I think this rule should default to a suggestion, but then be overwritten by our |
Add a ToDo comment to a list of avoided terms with a warning.
It should be case-insensitive (to cover all the variants: todo, TODO, ToDo).
The text was updated successfully, but these errors were encountered: