Remove line length lint warning
...because the current situation isn't helpful; no one looks at the list. Our style guide hasn't changed.
Ben Alpert committed
Jun 4, 2015 at 18:14 UTC
06de43d73e8f47e94a74f2b0e087ab6b0f84cca5
2 files changed
+5
-5
.eslintrc
+4
-4
@@ -43,14 +43,14 @@ rules:
43
no-shadow: 2
44
no-multi-spaces: 2
45
46
- # WARNINGS
47
- # This is the only one that's hard to track since we don't lint just changes.
48
- max-len: [1, 80]
49
-
46
# WISHLIST. One day...
47
# We'll need a custom version of this that does a subset of the whole rule.
48
# Otherwise this is just too noisy.
49
# valid-jsdoc: 1
50
+ # Ideally, we could just warn when *new* lines are added that exceed 80 chars
51
+ # while not warning about existing ones (often URLs, etc. which are
52
+ # necessarily long), but we don't have a good way to do so.
53
+ # max-len: [0, 80]
54
55
# DISABLED. These aren't compatible with our style
56
# We use this for private/internal variables
grunt/tasks/eslint.js
+1
-1
@@ -12,7 +12,7 @@ module.exports = function() {
12
if (err) {
13
grunt.log.error('Lint failed');
14
} else {
15
- grunt.log.ok('Lint passed (but may contain warnings)');
15
+ grunt.log.ok('Lint passed');
16
}
17
18
done(code === 0);