Merge pull request #533 from telerik/linter

Enable linter for common lib

Author: teobugslayer

Gruntfile.js

@@ -67,6 +67,17 @@ module.exports = function(grunt) {
 			}
 		},
 
+		tslint: {
+			build: {
+                files: {
+                    src: ["**/*.ts", "!node_modules/**/*.ts", "!messages/**/*.ts", "!**/*.d.ts"]
+                },
+                options: {
+                    configuration: grunt.file.readJSON("./tslint.json")
+                }
+            }
+        },
+
 		watch: {
 			devall: {
 				files: ["**/*.ts", "!node_modules/**/*.ts", 'test/**/*.ts'],
@@ -121,6 +132,7 @@ module.exports = function(grunt) {
 	grunt.loadNpmTasks("grunt-contrib-watch");
 	grunt.loadNpmTasks("grunt-shell");
 	grunt.loadNpmTasks("grunt-ts");
+	grunt.loadNpmTasks("grunt-tslint");
 
 	grunt.registerTask("set_package_version", function(version) {
 		var buildVersion = getBuildVersion(version);
@@ -167,6 +179,6 @@ module.exports = function(grunt) {
 		"shell:build_package",
 		"setPackageName"
 	]);
-
+	grunt.registerTask("lint", ["tslint:build"]);
 	grunt.registerTask("default", "ts:devlib");
 };

package.json

@@ -78,6 +78,7 @@
     "grunt-contrib-watch": "0.6.1",
     "grunt-shell": "1.1.2",
     "grunt-ts": "4.2.0",
+    "grunt-tslint": "2.5.0",
     "istanbul": "0.3.17",
     "mocha": "2.2.5",
     "mocha-fibers": "https://github.com/Icenium/mocha-fibers/tarball/master",

tslint.json

@@ -0,0 +1,49 @@
+{
+	"rules": {
+		"class-name": true,
+        "curly": true,
+		"eofline": true,
+        "indent": true,
+		"interface-name": true,
+        "jsdoc-format": true,
+        "max-line-length": [false, 140],
+        "no-consecutive-blank-lines": true,
+        "no-construct": true,
+        "no-debugger": true,
+        "no-duplicate-key": true,
+		"no-duplicate-variable": true,
+		"no-shadowed-variable": true,
+		"no-empty": true,
+        "no-eval": true,
+		"no-switch-case-fall-through": true,
+		"no-trailing-whitespace": true,
+		"no-unreachable": true,
+		"no-unused-expression": true,
+		"no-unused-variable": true,
+		"no-use-before-declare": true,
+		"no-var-keyword": true,
+		"no-var-requires": false,
+		"one-line": [
+			true,
+			"check-open-brace",
+			"check-catch",
+			"check-else"
+    	],
+		"quotemark": [false, "double"],
+		"semicolon": true,
+		"switch-default": false,
+        "triple-equals": [true, "allow-null-check"],
+		"use-strict": true,
+        "variable-name": [false, "allow-leading-underscore"],
+		"whitespace": [
+			false,
+			"check-branch",
+			"check-decl",
+			"check-operator",
+			"check-module",
+			"check-separator",
+			"check-type",
+			"check-typecast"
+		]
+	}
+}