-
Notifications
You must be signed in to change notification settings - Fork 35
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
Support disabling rules in jsdoc #174
Comments
👍 |
Thanks for report, ye, we should do it like an age ago. p.s. Would you like to implement this? I'm a little bit busy to do it completely by self but I can assist you. |
Hi @zxqfox I will give it ago. I am trying to compie the jscs files to this what should I look for settings wise to be able to do what this bug describes letting you enable and disable like you can in jscs. |
And where should I look. and what piece of code. |
@paladox JSCS passing options here: https://github.com/jscs-dev/jscs-jsdoc/blob/master/lib/rules/validate-jsdoc.js#L23 But I'd say it's somewhere in JSCS: https://github.com/jscs-dev/node-jscs/blob/master/lib/config/configuration.js#L67 |
Thanks. |
Sorry for long delay, but I still not sure where we should do it. ;-( I'm planning to try new jsdoctypeparser at wednesday and will look deeper into the problem. |
Thanks. |
duplicate of #61? |
Oh, not actually. This issue about disabling some jsDoc rules predefined in preset. Like if your google preset have some jsDoc settings and you need to disable just one, instead of copy-pasting the whole configuration. |
Please support disabling rules in jsdoc like you can in jscs.
For example a rule could look like
Since some preset set the default we should be allowed to override these like you can in jscs.
The text was updated successfully, but these errors were encountered: