Skip to content

Commit

Permalink
fix: remove forked indent rule (#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 authored and jthomerson committed Jan 22, 2020
1 parent 2aa0024 commit e246ade
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 2,193 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 e246ade

Please sign in to comment.