-
Notifications
You must be signed in to change notification settings - Fork 513
Add except
option for RequireCapitalizedComments
#1045
Comments
except
option for RequireCapitalizedComments
Great catch @ashclarke! We should totally do this. Would you like to give this a go? |
Shall I make jscs and jshint default exceptions? |
I reported this feature in the past #837, but completely forgot about it :-/ I am against making jscs and jshint (or any other values) default exceptions. Why? Because I don't want use inline comments for these two tools and want to get notified, when I do. |
@alawatthe I'd say no. It's a kind of sugar, atm we almost done #109 about presets, and I believe that it could helps a lot with sugar making in common. At other hand, I'm agree with @alawatthe about discouraging defaults. |
@zxqfox I am probably not seeing something obvious, but what has this bug to do with presets?
"No" to what? I'm a bit confused :-/ |
I have created a PR (#1052) with the change. This is one of my first contributions to an established repository, so please offer suggestions and advice if it is A) terrible or B) lacking something important! |
@alawatthe It was a message to @ashclarke, sorry. ;-) I'm talking about defaults, and presets can help with that. |
@zxqfox Ah, now I get it. No problem :-D |
Hello again!
This is a suggested feature to allow for certain exclusions to this rule, e.g:
istanbul | jshint - see Inline Configuration section | jscs
This allows hint comments to be ignored.
What do you think?
The text was updated successfully, but these errors were encountered: