Skip to content
This repository was archived by the owner on Mar 7, 2019. It is now read-only.

Commit 172ecc2

Browse files
Disable capitalized-comments πŸŽ‰
This rule has so far only caused pain everywhere - many a times you want to comment out a line of code to try something else. But this rule then fixes the first keyword on that line to be capitalised which, when uncommented, would cause a syntax error. Nobody wants that! Fixes #14
1 parent 1e706f3 commit 172ecc2

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

β€Žcoding-styles/fixable.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,6 @@ module.exports = {
5050
after: true,
5151
}],
5252

53-
// Enforce or disallow capitalization of the first letter of a comment
54-
'capitalized-comments': [1, 'always', {
55-
ignoreInlineComments: true,
56-
ignoreConsecutiveComments: true,
57-
}],
58-
5953
// Require Following Curly Brace Conventions
6054
// This rule is aimed at preventing bugs and increasing code clarity by ensuring that block
6155
// statements are wrapped in curly braces.

β€Žunused.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@ module.exports = {
1414
// Unused in favour of `no-var` rule. Front-end could have some interest in this rule.
1515
'block-scoped-var': 0,
1616

17+
// Enforce or disallow capitalization of the first letter of a comment
18+
// Unused, too restrictive.
19+
'capitalized-comments': 0,
20+
1721
// Limit Cyclomatic Complexity
1822
// Let the developer decide if what they write expresses their intent well enough, while
1923
// maintaining readability.

0 commit comments

Comments
Β (0)