feat(@angular/cli): extend tslnit config from recommended

no need to duplicate rules
new rules can be applied without updating, when new tslint is used

Fixes: angular#6179

Author: ValeryVS

packages/@angular/cli/blueprints/ng/files/__path__/polyfills.ts

@@ -18,7 +18,7 @@
  * BROWSER POLYFILLS
  */
 
-/** IE9, IE10 and IE11 requires all of the following polyfills. **/
+/** IE9, IE10 and IE11 requires all of the following polyfills. */
 // import 'core-js/es6/symbol';
 // import 'core-js/es6/object';
 // import 'core-js/es6/function';
@@ -40,23 +40,21 @@
 // import 'web-animations-js';  // Run `npm install --save web-animations-js`.
 
 
-/** Evergreen browsers require these. **/
+/** Evergreen browsers require these. */
 import 'core-js/es6/reflect';
 import 'core-js/es7/reflect';
 
 
-/** ALL Firefox browsers require the following to support `@angular/animation`. **/
+/** ALL Firefox browsers require the following to support `@angular/animation`. */
 // import 'web-animations-js';  // Run `npm install --save web-animations-js`.
 
 
-
 /***************************************************************************************************
  * Zone JS is required by Angular itself.
  */
 import 'zone.js/dist/zone';  // Included with Angular CLI.
 
 
-
 /***************************************************************************************************
  * APPLICATION IMPORTS
  */

packages/@angular/cli/blueprints/ng/files/__path__/test.ts

@@ -17,7 +17,7 @@ declare const __karma__: any;
 declare const require: any;
 
 // Prevent Karma from running prematurely.
-__karma__.loaded = function () {};
+__karma__.loaded = () => { };
 
 // First, initialize the Angular testing environment.
 getTestBed().initTestEnvironment(

packages/@angular/cli/blueprints/ng/files/e2e/app.e2e-spec.ts

@@ -7,10 +7,10 @@ describe('<%= htmlComponentName %> App', () => {
     page = new <%= jsComponentName %>Page();
   });
 
-  it('should display welcome message', done => {
+  it('should display welcome message', (done) => {
     page.navigateTo();
     page.getParagraphText()
-      .then(msg => expect(msg).toEqual('Welcome to <%= prefix %>!!'))
+      .then((msg) => expect(msg).toEqual('Welcome to <%= prefix %>!!'))
       .then(done, done.fail);
   });
 });

packages/@angular/cli/blueprints/ng/files/tslint.json

@@ -1,29 +1,16 @@
 {
-  "rulesDirectory": [
-    "node_modules/codelyzer"
+  "extends": [
+    "tslint:recommended"
   ],
   "rules": {
-    "arrow-return-shorthand": true,
-    "callable-types": true,
-    "class-name": true,
-    "comment-format": [
-      true,
-      "check-space"
-    ],
-    "curly": true,
-    "eofline": true,
-    "forin": true,
     "import-blacklist": [
       true,
       "rxjs"
     ],
-    "import-spacing": true,
-    "indent": [
+    "interface-name": [
       true,
-      "spaces"
+      "never-prefix"
     ],
-    "interface-over-type-literal": true,
-    "label-position": true,
     "max-line-length": [
       true,
       140
@@ -34,8 +21,6 @@
       "static-before-instance",
       "variables-before-functions"
     ],
-    "no-arg": true,
-    "no-bitwise": true,
     "no-console": [
       true,
       "debug",
@@ -44,69 +29,36 @@
       "timeEnd",
       "trace"
     ],
-    "no-construct": true,
-    "no-debugger": true,
-    "no-duplicate-super": true,
+    "no-consecutive-blank-lines": [
+      true,
+      2
+    ],
     "no-empty": false,
-    "no-empty-interface": true,
-    "no-eval": true,
     "no-inferrable-types": [
       true,
       "ignore-params"
     ],
-    "no-misused-new": true,
     "no-non-null-assertion": true,
-    "no-shadowed-variable": true,
     "no-string-literal": false,
-    "no-string-throw": true,
     "no-switch-case-fall-through": true,
-    "no-trailing-whitespace": true,
-    "no-unnecessary-initializer": true,
-    "no-unused-expression": true,
+    "no-trailing-whitespace": [
+      true,
+      "ignore-template-strings"
+    ],
     "no-use-before-declare": true,
-    "no-var-keyword": true,
     "object-literal-sort-keys": false,
-    "one-line": [
-      true,
-      "check-open-brace",
-      "check-catch",
-      "check-else",
-      "check-whitespace"
+    "ordered-imports": [
+      false
     ],
-    "prefer-const": true,
     "quotemark": [
       true,
-      "single"
+      "single",
+      "avoid-escape"
     ],
-    "radix": true,
-    "semicolon": [
-      "always"
+    "trailing-comma": [
+      false
     ],
-    "triple-equals": [
-      true,
-      "allow-null-check"
-    ],
-    "typedef-whitespace": [
-      true,
-      {
-        "call-signature": "nospace",
-        "index-signature": "nospace",
-        "parameter": "nospace",
-        "property-declaration": "nospace",
-        "variable-declaration": "nospace"
-      }
-    ],
-    "typeof-compare": true,
-    "unified-signatures": true,
     "variable-name": false,
-    "whitespace": [
-      true,
-      "check-branch",
-      "check-decl",
-      "check-operator",
-      "check-separator",
-      "check-type"
-    ],
     "directive-selector": [
       true,
       "attribute",
@@ -131,5 +83,8 @@
     "no-access-missing-member": true,
     "templates-use-public": true,
     "invoke-injectable": true
-  }
+  },
+  "rulesDirectory": [
+    "node_modules/codelyzer"
+  ]
 }

tests/e2e/tests/lint/lint-with-type-check-fail.ts

@@ -30,8 +30,7 @@ function check(val: any, fxState: any) {
     fxState.num = val + "";
   }
 }
-
-  `;
+`;
 
   return Promise.resolve()
     .then(() => writeFile(fileName, fileContents))