Skip to content

Commit

Permalink
fix: remove forked indent rule (silvermine#41)
Browse files Browse the repository at this point in the history
The reason for forking the indent rule was to allow us to line up
variable declarations. The original indent rule only allowed doing this
if you used 2 or 4 space indent. The new eslint rule has a setting to
require variable declarations to line up, so this rule is no longer
needed. In fact, our own config now uses the default rule.
  • Loading branch information
jimjenkins5 committed Dec 27, 2019
1 parent 2eac37f commit 34498d2
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 2,119 deletions.
4 changes: 1 addition & 3 deletions lib/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,7 @@ var fluentChaining = require('./rules/fluent-chaining'),
modulesOnly = require('./rules/module-files-only'),
blockScopeCase = require('./rules/block-scope-case'),
braceStyle = require('./rules/brace-style'),
maxStatementsPerLine = require('./rules/max-statements-per-line'),
indent = require('./rules/indent');
maxStatementsPerLine = require('./rules/max-statements-per-line');

module.exports.rules = {
'fluent-chaining': fluentChaining,
Expand All @@ -27,7 +26,6 @@ module.exports.rules = {
'no-multiple-inline-functions': noMultipleInlineFunctions,
'no-multiline-var-declarations': noMultilineVarDeclaration,
'brace-style': braceStyle,
'indent': indent,
'no-multiline-conditionals': noMultilineConditionals,
'empty-object-spacing': emptyObjectSpacing,
'empty-array-spacing': emptyArraySpacing,
Expand Down
Loading

0 comments on commit 34498d2

Please sign in to comment.