summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Krotscheck <krotscheck@gmail.com>2016-03-16 07:34:04 -0700
committerMichael Krotscheck <krotscheck@gmail.com>2016-03-16 08:08:50 -0700
commit5bc48bf279d2fb7d1995a614b2ec0809f5696f78 (patch)
treeb7791981b7adb7f6a7a3df549bde9bd650a6a4c2
parented866bd401b9b58d27785883c5110c0603699215 (diff)
Upgraded eslint to 2.4.0
This upgrades the eslint version to 2.4.0, removes zombie rules, and ensures that all new rules are disabled. Change-Id: I6249663f8e226c5fa2739481b4c79625f94c7eb5
Notes
Notes (review): Code-Review+2: Cindy Lu <clu@us.ibm.com> Workflow+1: Cindy Lu <clu@us.ibm.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Wed, 16 Mar 2016 18:28:10 +0000 Reviewed-on: https://review.openstack.org/293468 Project: openstack/eslint-config-openstack Branch: refs/heads/master
-rw-r--r--.eslintrc106
-rw-r--r--package.json4
2 files changed, 89 insertions, 21 deletions
diff --git a/.eslintrc b/.eslintrc
index e8c2a1f..b3856f2 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -183,6 +183,10 @@ rules:
183 # http://eslint.org/docs/rules/accessor-pairs 183 # http://eslint.org/docs/rules/accessor-pairs
184 accessor-pairs: 2 184 accessor-pairs: 2
185 185
186 # Enforces return statements in callbacks of array’s methods
187 # http://eslint.org/docs/rules/array-callback-return
188 array-callback-return: 0
189
186 # Treat var statements as if they were block scoped 190 # Treat var statements as if they were block scoped
187 # http://eslint.org/docs/rules/block-scoped-var 191 # http://eslint.org/docs/rules/block-scoped-var
188 block-scoped-var: 2 192 block-scoped-var: 2
@@ -241,9 +245,9 @@ rules:
241 # http://eslint.org/docs/rules/no-else-return 245 # http://eslint.org/docs/rules/no-else-return
242 no-else-return: 0 246 no-else-return: 0
243 247
244 # Disallow use of labels for anything other then loops and switches 248 # Disallow use of empty functions
245 # http://eslint.org/docs/rules/no-empty-label 249 # http://eslint.org/docs/rules/no-empty-function
246 no-empty-label: 2 250 no-empty-function: 0
247 251
248 # disallow use of empty destructuring patterns 252 # disallow use of empty destructuring patterns
249 # http://eslint.org/docs/rules/no-empty-pattern 253 # http://eslint.org/docs/rules/no-empty-pattern
@@ -265,6 +269,10 @@ rules:
265 # http://eslint.org/docs/rules/no-extra-bind 269 # http://eslint.org/docs/rules/no-extra-bind
266 no-extra-bind: 2 270 no-extra-bind: 2
267 271
272 # disallow unnecessary labels
273 # http://eslint.org/docs/rules/no-extra-label
274 no-extra-label: 0
275
268 # Disallow fallthrough of case statements 276 # Disallow fallthrough of case statements
269 # http://eslint.org/docs/rules/no-fallthrough 277 # http://eslint.org/docs/rules/no-fallthrough
270 no-fallthrough: 2 278 no-fallthrough: 2
@@ -277,6 +285,10 @@ rules:
277 # http://eslint.org/docs/rules/no-implicit-coercion 285 # http://eslint.org/docs/rules/no-implicit-coercion
278 no-implicit-coercion: 0 286 no-implicit-coercion: 0
279 287
288 # disallow var and named functions in global scope
289 # http://eslint.org/docs/rules/no-implicit-globals
290 no-implicit-globals: 0
291
280 # Disallow use of eval()-like methods 292 # Disallow use of eval()-like methods
281 # http://eslint.org/docs/rules/no-implied-eval 293 # http://eslint.org/docs/rules/no-implied-eval
282 no-implied-eval: 2 294 no-implied-eval: 2
@@ -365,6 +377,10 @@ rules:
365 # http://eslint.org/docs/rules/no-script-url 377 # http://eslint.org/docs/rules/no-script-url
366 no-script-url: 2 378 no-script-url: 2
367 379
380 # Disallow assignments where both sides are exactly the same
381 # http://eslint.org/docs/rules/no-self-assign
382 no-self-assign: 0
383
368 # Disallow comparisons where both sides are exactly the same 384 # Disallow comparisons where both sides are exactly the same
369 # http://eslint.org/docs/rules/no-self-compare 385 # http://eslint.org/docs/rules/no-self-compare
370 no-self-compare: 2 386 no-self-compare: 2
@@ -377,10 +393,18 @@ rules:
377 # http://eslint.org/docs/rules/no-throw-literal 393 # http://eslint.org/docs/rules/no-throw-literal
378 no-throw-literal: 2 394 no-throw-literal: 2
379 395
396 # disallow unmodified conditions of loops
397 # http://eslint.org/docs/rules/no-unmodified-loop-condition
398 no-unmodified-loop-condition: 0
399
380 # Disallow usage of expressions in statement position 400 # Disallow usage of expressions in statement position
381 # http://eslint.org/docs/rules/no-unused-expressions 401 # http://eslint.org/docs/rules/no-unused-expressions
382 no-unused-expressions: 2 402 no-unused-expressions: 2
383 403
404 # disallow unused labels
405 # http://eslint.org/docs/rules/no-unused-labels
406 no-unused-labels: 0
407
384 # disallow unnecessary .call() and .apply() 408 # disallow unnecessary .call() and .apply()
385 # http://eslint.org/docs/rules/no-useless-call 409 # http://eslint.org/docs/rules/no-useless-call
386 no-useless-call: 0 410 no-useless-call: 0
@@ -455,6 +479,10 @@ rules:
455 # http://eslint.org/docs/rules/no-label-var 479 # http://eslint.org/docs/rules/no-label-var
456 no-label-var: 2 480 no-label-var: 2
457 481
482 # restrict usage of specified global variables
483 # http://eslint.org/docs/rules/no-restricted-globals
484 no-restricted-globals: 0
485
458 # Disallow shadowing of names such as arguments 486 # Disallow shadowing of names such as arguments
459 # http://eslint.org/docs/rules/no-shadow-restricted-names 487 # http://eslint.org/docs/rules/no-shadow-restricted-names
460 no-shadow-restricted-names: 2 488 no-shadow-restricted-names: 2
@@ -578,6 +606,10 @@ rules:
578 # http://eslint.org/docs/rules/func-style 606 # http://eslint.org/docs/rules/func-style
579 func-style: 0 607 func-style: 0
580 608
609 # blacklist certain identifiers to prevent them being used
610 # http://eslint.org/docs/rules/id-blacklist
611 id-blacklist: 0
612
581 # this option enforces minimum and maximum identifier lengths (variable names, property names...) 613 # this option enforces minimum and maximum identifier lengths (variable names, property names...)
582 # http://eslint.org/docs/rules/id-length 614 # http://eslint.org/docs/rules/id-length
583 id-length: 0 615 id-length: 0
@@ -601,6 +633,10 @@ rules:
601 # http://eslint.org/docs/rules/key-spacing 633 # http://eslint.org/docs/rules/key-spacing
602 key-spacing: 0 634 key-spacing: 0
603 635
636 # enforce spacing before and after keywords
637 # http://eslint.org/docs/rules/keyword-spacing
638 keyword-spacing: 0
639
604 # Enforces empty lines around comments 640 # Enforces empty lines around comments
605 # http://eslint.org/docs/rules/lines-around-comment 641 # http://eslint.org/docs/rules/lines-around-comment
606 lines-around-comment: 0 642 lines-around-comment: 0
@@ -629,6 +665,14 @@ rules:
629 # http://eslint.org/docs/rules/newline-after-var 665 # http://eslint.org/docs/rules/newline-after-var
630 newline-after-var: 0 666 newline-after-var: 0
631 667
668 # require newline before return statement
669 # http://eslint.org/docs/rules/newline-before-return
670 newline-before-return: 0
671
672 # enforce newline after each call when chaining the calls
673 # http://eslint.org/docs/rules/newline-per-chained-call
674 newline-per-chained-call: 0
675
632 # Disallow use of the Array constructor 676 # Disallow use of the Array constructor
633 # http://eslint.org/docs/rules/no-array-constructor 677 # http://eslint.org/docs/rules/no-array-constructor
634 no-array-constructor: 0 678 no-array-constructor: 0
@@ -691,6 +735,10 @@ rules:
691 # http://eslint.org/docs/rules/no-unneeded-ternary 735 # http://eslint.org/docs/rules/no-unneeded-ternary
692 no-unneeded-ternary: 2 736 no-unneeded-ternary: 2
693 737
738 # disallow whitespace before properties
739 # http://eslint.org/docs/rules/no-whitespace-before-property
740 no-whitespace-before-property: 0
741
694 # Require or disallow padding inside curly braces 742 # Require or disallow padding inside curly braces
695 # http://eslint.org/docs/rules/object-curly-spacing 743 # http://eslint.org/docs/rules/object-curly-spacing
696 object-curly-spacing: 0 744 object-curly-spacing: 0
@@ -702,6 +750,10 @@ rules:
702 - uninitialized: "always" 750 - uninitialized: "always"
703 initialized: "never" 751 initialized: "never"
704 752
753 # require or disallow an newline around variable declarations
754 # http://eslint.org/docs/rules/one-var-declaration-per-line
755 one-var-declaration-per-line: 0
756
705 # Prevent assignment operator shorthand where possible 757 # Prevent assignment operator shorthand where possible
706 # http://eslint.org/docs/rules/operator-assignment 758 # http://eslint.org/docs/rules/operator-assignment
707 operator-assignment: 759 operator-assignment:
@@ -737,6 +789,10 @@ rules:
737 - before: false 789 - before: false
738 after: true 790 after: true
739 791
792 # sort import declarations within module
793 # http://eslint.org/docs/rules/sort-imports
794 sort-imports: 0
795
740 # Require or disallow use of semicolons instead of ASI 796 # Require or disallow use of semicolons instead of ASI
741 # http://eslint.org/docs/rules/semi 797 # http://eslint.org/docs/rules/semi
742 semi: 798 semi:
@@ -747,14 +803,6 @@ rules:
747 # http://eslint.org/docs/rules/sort-vars 803 # http://eslint.org/docs/rules/sort-vars
748 sort-vars: 0 804 sort-vars: 0
749 805
750 # Require a space after certain keywords
751 # http://eslint.org/docs/rules/space-after-keywords
752 space-after-keywords: 2
753
754 # Require a space before certain keywords
755 # http://eslint.org/docs/rules/space-before-keywords
756 space-before-keywords: 0
757
758 # require or disallow space before blocks 806 # require or disallow space before blocks
759 # http://eslint.org/docs/rules/space-before-blocks 807 # http://eslint.org/docs/rules/space-before-blocks
760 space-before-blocks: 808 space-before-blocks:
@@ -777,10 +825,6 @@ rules:
777 # http://eslint.org/docs/rules/space-infix-ops 825 # http://eslint.org/docs/rules/space-infix-ops
778 space-infix-ops: 2 826 space-infix-ops: 2
779 827
780 # Require a space after return, throw, and case
781 # http://eslint.org/docs/rules/space-return-throw-case
782 space-return-throw-case: 2
783
784 # Require or disallow spaces before/after unary operators (words on by default, nonwords) 828 # Require or disallow spaces before/after unary operators (words on by default, nonwords)
785 # http://eslint.org/docs/rules/space-unary-ops 829 # http://eslint.org/docs/rules/space-unary-ops
786 space-unary-ops: 830 space-unary-ops:
@@ -820,14 +864,14 @@ rules:
820 # http://eslint.org/docs/rules/generator-star-spacing 864 # http://eslint.org/docs/rules/generator-star-spacing
821 generator-star-spacing: 0 865 generator-star-spacing: 0
822 866
823 # disallow arrow functions where a condition is expected
824 # http://eslint.org/docs/rules/no-arrow-condition
825 no-arrow-condition: 0
826
827 # disallow modifying variables of class declarations 867 # disallow modifying variables of class declarations
828 # http://eslint.org/docs/rules/no-class-assign 868 # http://eslint.org/docs/rules/no-class-assign
829 no-class-assign: 0 869 no-class-assign: 0
830 870
871 # disallow arrow functions where they could be confused with comparisons
872 # http://eslint.org/docs/rules/no-confusing-arrow
873 no-confusing-arrow: 0
874
831 # disallow modifying variables that are declared using const 875 # disallow modifying variables that are declared using const
832 # http://eslint.org/docs/rules/no-const-assign 876 # http://eslint.org/docs/rules/no-const-assign
833 no-const-assign: 0 877 no-const-assign: 0
@@ -836,10 +880,22 @@ rules:
836 # http://eslint.org/docs/rules/no-dupe-class-members 880 # http://eslint.org/docs/rules/no-dupe-class-members
837 no-dupe-class-members: 0 881 no-dupe-class-members: 0
838 882
883 # disallow use of the new operator with the Symbol object
884 # http://eslint.org/docs/rules/no-new-symbol
885 no-new-symbol: 0
886
887 # restrict usage of specified modules when loaded by import declaration
888 # http://eslint.org/docs/rules/no-restricted-imports
889 no-restricted-imports: 0
890
839 # disallow use of this/super before calling super() in constructors 891 # disallow use of this/super before calling super() in constructors
840 # http://eslint.org/docs/rules/no-this-before-super 892 # http://eslint.org/docs/rules/no-this-before-super
841 no-this-before-super: 0 893 no-this-before-super: 0
842 894
895 # disallow unnecessary constructor
896 # http://eslint.org/docs/rules/no-useless-constructor
897 no-useless-constructor: 0
898
843 # require let or const instead of var 899 # require let or const instead of var
844 # http://eslint.org/docs/rules/no-var 900 # http://eslint.org/docs/rules/no-var
845 no-var: 0 901 no-var: 0
@@ -864,6 +920,10 @@ rules:
864 # http://eslint.org/docs/rules/prefer-reflect 920 # http://eslint.org/docs/rules/prefer-reflect
865 prefer-reflect: 0 921 prefer-reflect: 0
866 922
923 # suggest using the rest parameters instead of arguments
924 # http://eslint.org/docs/rules/prefer-rest-params
925 prefer-rest-params: 0
926
867 # suggest using template literals instead of strings concatenation 927 # suggest using template literals instead of strings concatenation
868 # http://eslint.org/docs/rules/prefer-template 928 # http://eslint.org/docs/rules/prefer-template
869 prefer-template: 0 929 prefer-template: 0
@@ -872,6 +932,14 @@ rules:
872 # http://eslint.org/docs/rules/require-yield 932 # http://eslint.org/docs/rules/require-yield
873 require-yield: 0 933 require-yield: 0
874 934
935 # enforce spacing around embedded expressions of template strings
936 # http://eslint.org/docs/rules/template-curly-spacing
937 template-curly-spacing: 0
938
939 # enforce spacing around the * in yield* expressions
940 # http://eslint.org/docs/rules/yield-star-spacing
941 yield-star-spacing: 0
942
875 ############################################################################# 943 #############################################################################
876 # Legacy 944 # Legacy
877 ############################################################################# 945 #############################################################################
diff --git a/package.json b/package.json
index e2d10aa..8054cbb 100644
--- a/package.json
+++ b/package.json
@@ -29,10 +29,10 @@
29 }, 29 },
30 "homepage": "https://wiki.openstack.org/wiki/QA", 30 "homepage": "https://wiki.openstack.org/wiki/QA",
31 "peerDependencies": { 31 "peerDependencies": {
32 "eslint": "^1.10.3" 32 "eslint": "^2.4.0"
33 }, 33 },
34 "devDependencies": { 34 "devDependencies": {
35 "eslint": "1.10.3", 35 "eslint": "2.4.0",
36 "jasmine": "2.3.1" 36 "jasmine": "2.3.1"
37 }, 37 },
38 "dependencies": { 38 "dependencies": {