Coverage #5
Coverage #5
Conversation
| "unused": true, | ||
| "strict": true, | ||
| "node": true, | ||
| "latedef": true, |
There was a problem hiding this comment.
why did you remove the comments?
There was a problem hiding this comment.
Because comments are not valid json, and a jsonlinter gets angry at it. Don't you have a jsonlinter in sublime? :p
Also, all the comments were doing is repeating what the setting was. Writing 2, and then two again in longer English, isn't really what comments are for. If they were saying why it's 2, then I'd be much more interested.
There was a problem hiding this comment.
yeah, but a .jshintrc technically isn't json.
I agree that "how" comments are bad and wrong, but some of them are "why" comments. Also, I really like knowing what eqeqeq and newcap do without having to look them up every. single. time.
(this is especially important with jshint, since half the rules are positive (allow these things) and half the rules are negative (ban these other things).
|
Changes Unknown when pulling 36ce898 on coverage into * on master*. |
|
Unknown, eh? That's pretty spooky. |
|
That is spooky. At least its reporting coverage correctly. I'll add back the comments. Maybe someday they'll allow yaml. |
|
Now I have this stuck in my head, thanks. |
|
Changes Unknown when pulling ab2ef3d on coverage into * on master*. |
Add coverage reporting.