Added tslint for editor and webpack

Refs #20

Author: sverhoeven

packages/react-scripts/config/webpack.config.dev.js

@@ -101,8 +101,13 @@ module.exports = {
   // @remove-on-eject-end
   module: {
     // First, run the linter.
-    // It's important to do this before Babel processes the JS.
+    // It's important to do this before Babel or TypeScript processes the JS/TS.
     preLoaders: [
+      {
+        test: /\.(ts|tsx)$/,
+        loader: 'tslint',
+        include: paths.appSrc
+      },
       {
         test: /\.(js|jsx)$/,
         loader: 'eslint',
@@ -112,7 +117,7 @@ module.exports = {
     loaders: [
       // Process TS with TypeScript and then with Babel
       {
-        test: /\.(ts|tsx)?$/,
+        test: /\.(ts|tsx)$/,
         include: paths.appSrc,
         loader: 'awesome-typescript',
         query: {

packages/react-scripts/config/webpack.config.prod.js

@@ -107,8 +107,13 @@ module.exports = {
   // @remove-on-eject-end
   module: {
     // First, run the linter.
-    // It's important to do this before Babel processes the JS.
+    // It's important to do this before Babel or TypeScript processes the JS/TS.
     preLoaders: [
+      {
+        test: /\.(ts|tsx)$/,
+        loader: 'tslint',
+        include: paths.appSrc
+      },
       {
         test: /\.(js|jsx)$/,
         loader: 'eslint',

packages/react-scripts/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nlesc/react-scripts",
-  "version": "0.0.8",
+  "version": "0.0.9",
   "description": "Configuration and scripts for Create React App with Typescript.",
   "repository": "NLeSC/create-react-app",
   "license": "BSD-3-Clause",
@@ -63,6 +63,9 @@
     "strip-ansi": "3.0.1",
     "style-loader": "0.13.1",
     "typescript": "^2.0.7",
+    "tslint": "^3.15.1",
+    "tslint-loader": "^2.1.5",
+    "tslint-react": "^1.1.0",
     "url-loader": "0.5.7",
     "webpack": "1.13.2",
     "webpack-dev-server": "1.16.2",

packages/react-scripts/template/tslint.json

@@ -0,0 +1,67 @@
+{
+    "extends": ["tslint:latest", "tslint-react"],
+    "rules": {
+        "class-name": true,
+        "comment-format": [
+            true,
+            "check-space"
+        ],
+        "indent": [
+            true,
+            "spaces"
+        ],
+        "jsx-alignment": true,
+        "jsx-no-lambda": false,
+        "jsx-no-string-ref": true,
+        "jsx-no-multiline-js": false,
+        "jsx-self-close": true,
+        "member-access": false,
+        "no-duplicate-variable": true,
+        "no-eval": true,
+        "no-internal-module": true,
+        "no-trailing-whitespace": true,
+        "no-unsafe-finally": true,
+        "no-var-keyword": true,
+        "one-line": [
+            true,
+            "check-open-brace",
+            "check-whitespace"
+        ],
+        "ordered-imports": false,
+        "quotemark": [
+            true,
+            "single",
+            "jsx-double"
+        ],
+        "semicolon": [
+            true,
+            "always"
+        ],
+        "triple-equals": [
+            true,
+            "allow-null-check"
+        ],
+        "typedef-whitespace": [
+            true,
+            {
+                "call-signature": "nospace",
+                "index-signature": "nospace",
+                "parameter": "nospace",
+                "property-declaration": "nospace",
+                "variable-declaration": "nospace"
+            }
+        ],
+        "variable-name": [
+            true,
+            "ban-keywords"
+        ],
+        "whitespace": [
+            true,
+            "check-branch",
+            "check-decl",
+            "check-operator",
+            "check-separator",
+            "check-type"
+        ]
+    }
+}