tools: alphabetize eslint stylistic issues section
authorRich Trott <rtrott@gmail.com>
Mon, 8 Feb 2016 04:57:14 +0000 (20:57 -0800)
committerMyles Borins <mborins@us.ibm.com>
Wed, 2 Mar 2016 22:01:11 +0000 (14:01 -0800)
Rearrange the style rules in .eslintrc to be in alphabetical order.

This has two benefits:

It means the rules appear in the same order as they do in the ESLint
documentation, easing cross-referencing.

It also means that it is much easier to determine with visual inspection
if a rule is set or not.

https://github.com/nodejs/node/pull/5135
Reviewed-By: Michaƫl Zasso <mic.besace@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
.eslintrc

index ef27556..24db5df 100644 (file)
--- a/.eslintrc
+++ b/.eslintrc
@@ -55,36 +55,36 @@ rules:
 
   # Stylistic Issues
   # list: https://github.com/eslint/eslint/tree/master/docs/rules#stylistic-issues
-  ## use single quote, we can use double quote when escape chars
-  quotes: [2, "single", "avoid-escape"]
-  ## 2 space indentation
-  indent: [2, 2, {SwitchCase: 1}]
   ## add space after comma
   comma-spacing: 2
-  ## put semi-colon
-  semi: 2
-  ## require spaces operator like var sum = 1 + 1;
-  space-infix-ops: 2
-  ## require spaces return, throw, case
-  space-return-throw-case: 2
-  ## no space before function, eg. 'function()'
-  space-before-function-paren: [2, "never"]
-  ## require space before blocks, eg 'function() {'
-  space-before-blocks: [2, "always"]
-  ## require parens for Constructor
-  new-parens: 2
+  ## require newline at end of files
+  eol-last: 2
+  ## 2 space indentation
+  indent: [2, 2, {SwitchCase: 1}]
   ## max 80 length
   max-len: [2, 80, 2]
+  ## require parens for Constructor
+  new-parens: 2
   ## max 2 consecutive empty lines
   no-multiple-empty-lines: [2, {max: 2}]
-  ## require newline at end of files
-  eol-last: 2
   ## no trailing spaces
   no-trailing-spaces: 2
+  ## use single quote, we can use double quote when escape chars
+  quotes: [2, "single", "avoid-escape"]
+  ## put semi-colon
+  semi: 2
   ## require space after keywords, eg 'for (..)'
   space-after-keywords: 2
+  ## require space before blocks, eg 'function() {'
+  space-before-blocks: [2, "always"]
+  ## no space before function, eg. 'function()'
+  space-before-function-paren: [2, "never"]
   ## no leading/trailing spaces in parens
   space-in-parens: [2, "never"]
+  ## require spaces operator like var sum = 1 + 1;
+  space-infix-ops: 2
+  ## require spaces return, throw, case
+  space-return-throw-case: 2
   ## no spaces with non-word unary operators, require for word unary operators
   space-unary-ops: 2