Merge pull request #337 from whyboris/codelyzer-bump

bump tslint & codelyzer versions, update tslint rules & alphabetize

package.json

@@ -74,7 +74,7 @@
     "rxjs": "6.5.2",
     "spectron": "5.0.0",
     "ts-node": "7.0.1",
-    "tslint": "5.11.0",
+    "tslint": "5.17.0",
     "typescript": "3.4.5",
     "wait-on": "3.2.0",
     "webdriver-manager": "12.1.0",

tslint.json

@@ -1,36 +1,23 @@
 {
-  "rulesDirectory": [
+  "rulesDirectory": ["node_modules/codelyzer"],
-    "node_modules/codelyzer"
-  ],
   "rules": {
     "arrow-return-shorthand": true,
     "callable-types": true,
     "class-name": true,
-    "comment-format": [
+    "comment-format": [true, "check-space"],
-      true,
+    "component-class-suffix": true,
-      "check-space"
+    "component-selector": [true, "element", "app", "kebab-case"],
-    ],
     "curly": true,
-    "deprecation": {
+    "deprecation": { "severity": "warn" },
-      "severity": "warn"
+    "directive-class-suffix": true,
-    },
     "eofline": true,
     "forin": true,
-    "import-blacklist": [
+    "import-blacklist": [true, "rxjs/Rx"],
-      true,
-      "rxjs/Rx"
-    ],
     "import-spacing": true,
-    "indent": [
+    "indent": [true, "spaces"],
-      true,
-      "spaces"
-    ],
     "interface-over-type-literal": true,
     "label-position": true,
-    "max-line-length": [
+    "max-line-length": [true, 140],
-      true,
-      140
-    ],
     "member-access": false,
     "member-ordering": [
       true,
@@ -45,26 +32,22 @@
     ],
     "no-arg": true,
     "no-bitwise": true,
-    "no-console": [
+    "no-console": [true, "debug", "info", "time", "timeEnd", "trace"],
-      true,
-      "debug",
-      "info",
-      "time",
-      "timeEnd",
-      "trace"
-    ],
     "no-construct": true,
     "no-debugger": true,
     "no-duplicate-super": true,
     "no-empty": false,
     "no-empty-interface": true,
     "no-eval": true,
-    "no-inferrable-types": [
+    "no-host-metadata-property": true,
-      true,
+    "no-inferrable-types": [true, "ignore-params"],
-      "ignore-params"
+    "no-input-rename": true,
-    ],
+    "no-inputs-metadata-property": true,
     "no-misused-new": true,
     "no-non-null-assertion": true,
+    "no-output-on-prefix": true,
+    "no-output-rename": true,
+    "no-outputs-metadata-property": true,
     "no-shadowed-variable": true,
     "no-string-literal": false,
     "no-string-throw": true,
@@ -72,7 +55,6 @@
     "no-trailing-whitespace": true,
     "no-unnecessary-initializer": true,
     "no-unused-expression": true,
-    "no-use-before-declare": true,
     "no-var-keyword": true,
     "object-literal-sort-keys": false,
     "one-line": [
@@ -83,19 +65,10 @@
       "check-whitespace"
     ],
     "prefer-const": true,
-    "quotemark": [
+    "quotemark": [true, "single"],
-      true,
-      "single"
-    ],
     "radix": true,
-    "semicolon": [
+    "semicolon": [true, "always"],
-      true,
+    "triple-equals": [true, "allow-null-check"],
-      "always"
-    ],
-    "triple-equals": [
-      true,
-      "allow-null-check"
-    ],
     "typedef-whitespace": [
       true,
       {
@@ -107,6 +80,9 @@
       }
     ],
     "unified-signatures": true,
+    "use-life-cycle-interface": true,
+    "use-lifecycle-interface": true,
+    "use-pipe-transform-interface": true,
     "variable-name": false,
     "whitespace": [
       true,
@@ -115,22 +91,6 @@
       "check-operator",
       "check-separator",
       "check-type"
-    ],
+    ]
-    "component-selector": [
-      true,
-      "element",
-      "app",
-      "kebab-case"
-    ],
-    "no-output-on-prefix": true,
-    "no-inputs-metadata-property": true,
-    "no-outputs-metadata-property": true,
-    "no-host-metadata-property": true,
-    "no-input-rename": true,
-    "no-output-rename": true,
-    "use-lifecycle-interface": true,
-    "use-pipe-transform-interface": true,
-    "component-class-suffix": true,
-    "directive-class-suffix": true
   }
 }