jscs-dev / jscs-jsdoc

JsDoc validation rules for jscs
MIT License
99 stars 35 forks source link

Support disabling rules in jsdoc #174

Open paladox opened 8 years ago

paladox commented 8 years ago

Please support disabling rules in jsdoc like you can in jscs.

For example a rule could look like

"jsDoc": {
    "requireParamTypes": null
}

Since some preset set the default we should be allowed to override these like you can in jscs.

ricordisamoa commented 8 years ago

:+1: Of course it should merge jsDoc rules from presets.

qfox commented 8 years ago

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.

paladox commented 8 years ago

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.

paladox commented 8 years ago

And where should I look. and what piece of code.

qfox commented 8 years ago

@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

paladox commented 8 years ago

Thanks.

qfox commented 8 years ago

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.

paladox commented 8 years ago

Thanks.

elijahsmith commented 8 years ago

duplicate of #61?

qfox commented 8 years ago

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.