Skip to content

Commit

Permalink
tools: remove legacy indentation linting
Browse files Browse the repository at this point in the history
All linting now uses the current ESLint 4.3.0 indentation linting.
Remove legacy indentation rules.

PR-URL: #14515
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
  • Loading branch information
Trott authored and addaleax committed Aug 1, 2017
1 parent b41ae98 commit fa8c5f4
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 67 deletions.
19 changes: 7 additions & 12 deletions .eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -101,18 +101,13 @@ rules:
func-call-spacing: error
func-name-matching: error
func-style: [error, declaration, {allowArrowFunctions: true}]
# indent: [error, 2, {ArrayExpression: first,
# CallExpression: {arguments: first},
# FunctionDeclaration: {parameters: first},
# FunctionExpression: {parameters: first},
# MemberExpression: off,
# ObjectExpression: first,
# SwitchCase: 1}]
indent-legacy: [error, 2, {ArrayExpression: first,
CallExpression: {arguments: first},
MemberExpression: 1,
ObjectExpression: first,
SwitchCase: 1}]
indent: [error, 2, {ArrayExpression: first,
CallExpression: {arguments: first},
FunctionDeclaration: {parameters: first},
FunctionExpression: {parameters: first},
MemberExpression: off,
ObjectExpression: first,
SwitchCase: 1}]
key-spacing: [error, {mode: minimum}]
keyword-spacing: error
linebreak-style: [error, unix]
Expand Down
13 changes: 0 additions & 13 deletions benchmark/.eslintrc.yaml

This file was deleted.

10 changes: 0 additions & 10 deletions doc/.eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,3 @@ rules:
no-var: error
prefer-const: error
prefer-rest-params: error

# use stricter indent over indent-legacy
indent-legacy: off
indent: [error, 2, {ArrayExpression: first,
CallExpression: {arguments: first},
FunctionDeclaration: {parameters: first},
FunctionExpression: {parameters: first},
MemberExpression: off,
ObjectExpression: first,
SwitchCase: 1}]
9 changes: 0 additions & 9 deletions lib/.eslintrc.yaml
Original file line number Diff line number Diff line change
@@ -1,13 +1,4 @@
rules:
indent: [error, 2, {ArrayExpression: first,
CallExpression: {arguments: first},
FunctionDeclaration: {parameters: first},
FunctionExpression: {parameters: first},
MemberExpression: off,
ObjectExpression: first,
SwitchCase: 1}]
indent-legacy: off

# Custom rules in tools/eslint-rules
require-buffer: error
buffer-constructor: error
Expand Down
10 changes: 0 additions & 10 deletions test/.eslintrc.yaml
Original file line number Diff line number Diff line change
@@ -1,16 +1,6 @@
## Test-specific linter rules

rules:
# Stylistic Issues
# http://eslint.org/docs/rules/#stylistic-issues
indent: [error, 2, {ArrayExpression: first,
CallExpression: {arguments: first},
FunctionDeclaration: {parameters: first},
FunctionExpression: {parameters: first},
MemberExpression: off,
ObjectExpression: first,
SwitchCase: 1}]
indent-legacy: off
# ECMAScript 6
# http://eslint.org/docs/rules/#ecmascript-6
no-var: error
Expand Down
13 changes: 0 additions & 13 deletions tools/.eslintrc.yaml

This file was deleted.

0 comments on commit fa8c5f4

Please sign in to comment.