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: nodejs#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>
Trott committed Jul 30, 2017
1 parent 2791761 commit e9b67f7
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
@@ -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]
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
@@ -11,13 +11,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
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
13 changes: 0 additions & 13 deletions tools/.eslintrc.yaml

This file was deleted.

0 comments on commit e9b67f7

Please sign in to comment.