tslint update added no-unused-variable, check-decl rules

b091 · Sep 7, 2015 · 4f8a6c7 · 4f8a6c7
1 parent a521060
commit 4f8a6c7
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -15,7 +15,7 @@
     "gulp-jshint": "^1.11.2",
     "gulp-protractor": "^1.0.0",
     "gulp-replace": "^0.5.4",
-    "gulp-tslint": "^3.1.2",
+    "gulp-tslint": "^3.2.0",
     "gulp-typedoc": "^1.2.1",
     "gulp-webserver": "^0.9.1",
     "jshint-stylish": "^2.0.1",

diff --git a/src/module/story/StoryService.ts b/src/module/story/StoryService.ts
@@ -1,4 +1,3 @@
-import {App} from "../../App";
 import {EndpointService} from "../../services/EndpointService";
 
 export class StoryService {

diff --git a/tasks/tslint.js b/tasks/tslint.js
@@ -5,7 +5,8 @@ module.exports = function(gulp, directory, configuration) {
         var tslint = require('gulp-tslint');
 
         return gulp.src([
-            path.join(directory, '**', '*.ts')
+            path.join(directory, '**', '*.ts'),
+            path.join('!', directory, 'references.ts')
         ]).pipe(tslint({
             configuration: require(configuration)
         })).pipe(tslint.report('verbose'));

diff --git a/test/references.ts b/test/references.ts
@@ -5,6 +5,8 @@
 /// <reference path="../vendor/typings/angular-protractor/angular-protractor.d.ts" />
 
 // protractor missing declaration
+// noinspection TsLint
 var startLocalhost:any;
 // protractor..
+// noinspection TsLint
 var expect:any;
diff --git a/tslint.json b/tslint.json
@@ -37,7 +37,7 @@
     "no-switch-case-fall-through": true,
     "no-trailing-comma": true,
     "no-trailing-whitespace": true,
-    "no-unused-variable": false,
+    "no-unused-variable": true,
     "no-unreachable": true,
     "no-var-requires": true,
     "one-line": [true,
@@ -71,6 +71,7 @@
     ],
     "variable-name": true,
     "whitespace": [true,
+      "check-decl",
       "check-branch",
       "check-operator",
       "check-separator"