-
Notifications
You must be signed in to change notification settings - Fork 9
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 rule rrd amount of comments #273
Comments
If more than 5% of the total lines are comments, trigger a warning? @rrd108 |
Yep. Or reather character count as comments can be in the same line as the code. What do you think? |
In that case would 5% be a good number? should it be higher or lower? |
Let's try with this and check a few files and then we can discuss |
jsdoc comments should be ignored |
I want to keep helping with bugs so Ill unasign myself from this |
I created a PR for this #377 |
feat: add new rrd amount of comments rule w tests (#273)
We should check how much comments we have in a file and if it too much (without jsdocs) we should fire a warning
Any number we decide will be debatablee. My suggestion is more than 5%
The text was updated successfully, but these errors were encountered: