typescript-eslint
diff --git a/‎.eslintrc.json
+3-16 b/‎.eslintrc.json
+3-16
diff --git a/‎.gitignore
-3 b/‎.gitignore
-3
diff --git a/‎.vscode/launch.json
+3-2 b/‎.vscode/launch.json
+3-2
diff --git a/‎azure-pipelines.yml
+6 b/‎azure-pipelines.yml
+6
diff --git a/‎package.json
+2-1 b/‎package.json
+2-1
diff --git a/‎packages/eslint-plugin-tslint/package.json
+3-2 b/‎packages/eslint-plugin-tslint/package.json
+3-2
diff --git a/‎packages/eslint-plugin-tslint/tsconfig.build.json
+7 b/‎packages/eslint-plugin-tslint/tsconfig.build.json
+7
diff --git a/‎packages/eslint-plugin-tslint/tsconfig.json
+3-6 b/‎packages/eslint-plugin-tslint/tsconfig.json
+3-6
diff --git a/‎packages/eslint-plugin/.npmignore
-6 b/‎packages/eslint-plugin/.npmignore
-6
diff --git a/‎packages/eslint-plugin/jest.config.js
+2-2 b/‎packages/eslint-plugin/jest.config.js
+2-2
diff --git a/‎packages/eslint-plugin/lib/index.js
-25 b/‎packages/eslint-plugin/lib/index.js
-25
diff --git a/‎packages/eslint-plugin/lib/rules/adjacent-overload-signatures.js
-138 b/‎packages/eslint-plugin/lib/rules/adjacent-overload-signatures.js
-138
diff --git a/‎packages/eslint-plugin/lib/rules/class-name-casing.js
-96 b/‎packages/eslint-plugin/lib/rules/class-name-casing.js
-96
@@ -30,7 +30,7 @@
     {
       "files": [
         "packages/eslint-plugin-tslint/tests/**/*.ts",
-        "packages/eslint-plugin/tests/**/*.js",
+        "packages/eslint-plugin/tests/**/*.test.ts",
         "packages/parser/tests/**/*.ts",
         "packages/typescript-estree/tests/**/*.ts"
       ],
@@ -53,22 +53,9 @@
       }
     },
     {
-      "files": ["packages/eslint-plugin/**/*.js"],
+      "files": ["packages/eslint-plugin/test/**/*.ts"],
       "rules": {
-        "eslint-plugin/fixer-return": "error",
-        "eslint-plugin/no-identical-tests": "error",
-        "eslint-plugin/no-missing-placeholders": "error",
-        "eslint-plugin/no-unused-placeholders": "error",
-        "eslint-plugin/no-useless-token-range": "error",
-        "eslint-plugin/require-meta-fixable": "error",
-        "eslint-plugin/prefer-placeholders": "error",
-        "eslint-plugin/prefer-replace-text": "error",
-        "eslint-plugin/no-deprecated-report-api": "error",
-        "eslint-plugin/report-message-format": ["error", "^[A-Z'{].*[\\.}]$"],
-        "eslint-plugin/no-deprecated-context-methods": "error",
-        "eslint-plugin/prefer-output-null": "error",
-        "eslint-plugin/test-case-shorthand-strings": "error",
-        "eslint-plugin/require-meta-type": "error"
+        "eslint-plugin/no-identical-tests": "error"
       }
     }
   ]
 
@@ -36,9 +36,6 @@ build/Release
 node_modules/
 jspm_packages/
 
-# TypeScript v1 declaration files
-typings/
-
 # Optional npm cache directory
 .npm
 
 
@@ -11,9 +11,10 @@
           "cwd": "${workspaceFolder}/packages/eslint-plugin/",
           "program": "${workspaceFolder}/node_modules/jest/bin/jest.js",
           "args": [
-              "--colors",
-              "tests/lib/rules/${fileBasename}"
+              "--runInBand",
+              "tests/rules/${fileBasenameNoExtension}"
           ],
+          "sourceMaps": true,
           "console": "integratedTerminal",
           "internalConsoleOptions": "neverOpen"
       }
 
@@ -17,6 +17,12 @@ jobs:
           # bootstrap
           yarn --ignore-engines --frozen-lockfile
 
+      - script: |
+          # Note that this command *also* typechecks tests/tools,
+          # whereas the build only checks src files
+          yarn typecheck
+        displayName: 'Typecheck all packages'
+
       - script: |
           yarn check-format
         displayName: 'Check code formatting'
 
@@ -7,7 +7,7 @@
   "contributors": [
     "James Henry <typescript-eslint@jameshenry.blog>",
     "Nicholas C. Zakas",
-    "Brad Zacher",
+    "Brad Zacher <brad.zacher@gmail.com>",
     "armano2",
     "Jed Fox"
   ],
@@ -21,6 +21,7 @@
     "test": "lerna run test --parallel",
     "build": "lerna run build",
     "clean": "lerna clean && lerna run clean",
+    "typecheck": "lerna run typecheck",
     "lint": "eslint . --ext .js,.ts",
     "lint-fix": "eslint . --ext .js,.ts --fix",
     "cz": "git-cz",
 
@@ -21,8 +21,9 @@
   "scripts": {
     "test": "jest --coverage",
     "prebuild": "npm run clean",
-    "build": "tsc",
-    "clean": "rimraf dist/"
+    "build": "tsc -p tsconfig.build.json",
+    "clean": "rimraf dist/",
+    "typecheck": "tsc --noEmit"
   },
   "dependencies": {
     "lodash.memoize": "^4.1.2"
 
@@ -0,0 +1,7 @@
+{
+  "extends": "../../tsconfig.base.json",
+  "compilerOptions": {
+    "outDir": "./dist"
+  },
+  "include": ["src"]
+}
@@ -1,8 +1,5 @@
 {
-  "extends": "../../tsconfig.json",
-  "compilerOptions": {
-    "declaration": true,
-    "outDir": "./dist"
-  },
-  "include": ["src"]
+  "extends": "./tsconfig.build.json",
+  "include": ["src", "tests"],
+  "exclude": ["tests/test-project", "tests/test-tslint-rules-directory"]
 }
@@ -5,9 +5,9 @@ module.exports = {
   transform: {
     '^.+\\.tsx?$': 'ts-jest'
   },
-  testRegex: './tests/lib/.+\\.js$',
+  testRegex: './tests/.+\\.test\\.ts$',
   collectCoverage: false,
-  collectCoverageFrom: ['lib/**/*.{js,jsx,ts,tsx}'],
+  collectCoverageFrom: ['src/**/*.{js,jsx,ts,tsx}'],
   moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json', 'node'],
   coverageReporters: ['text-summary', 'lcov']
 };
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@`
`30`	`30`	`{`
`31`	`31`	`"files": [`
`32`	`32`	`"packages/eslint-plugin-tslint/tests/*/.ts",`
`33`		`- "packages/eslint-plugin/tests/*/.js",`
	`33`	`+ "packages/eslint-plugin/tests/*/.test.ts",`
`34`	`34`	`"packages/parser/tests/*/.ts",`
`35`	`35`	`"packages/typescript-estree/tests/*/.ts"`
`36`	`36`	`],`
`@@ -53,22 +53,9 @@`
`53`	`53`	`}`
`54`	`54`	`},`
`55`	`55`	`{`
`56`		`- "files": ["packages/eslint-plugin/*/.js"],`
	`56`	`+ "files": ["packages/eslint-plugin/test/*/.ts"],`
`57`	`57`	`"rules": {`
`58`		`- "eslint-plugin/fixer-return": "error",`
`59`		`- "eslint-plugin/no-identical-tests": "error",`
`60`		`- "eslint-plugin/no-missing-placeholders": "error",`
`61`		`- "eslint-plugin/no-unused-placeholders": "error",`
`62`		`- "eslint-plugin/no-useless-token-range": "error",`
`63`		`- "eslint-plugin/require-meta-fixable": "error",`
`64`		`- "eslint-plugin/prefer-placeholders": "error",`
`65`		`- "eslint-plugin/prefer-replace-text": "error",`
`66`		`- "eslint-plugin/no-deprecated-report-api": "error",`
`67`		`- "eslint-plugin/report-message-format": ["error", "^[A-Z'{].*[\\.}]$"],`
`68`		`- "eslint-plugin/no-deprecated-context-methods": "error",`
`69`		`- "eslint-plugin/prefer-output-null": "error",`
`70`		`- "eslint-plugin/test-case-shorthand-strings": "error",`
`71`		`- "eslint-plugin/require-meta-type": "error"`
	`58`	`+ "eslint-plugin/no-identical-tests": "error"`
`72`	`59`	`}`
`73`	`60`	`}`
`74`	`61`	`]`
Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,5 @@`
`1`	`1`	`{`
`2`		`- "extends": "../../tsconfig.json",`
`3`		`- "compilerOptions": {`
`4`		`- "declaration": true,`
`5`		`- "outDir": "./dist"`
`6`		`- },`
`7`		`- "include": ["src"]`
	`2`	`+ "extends": "./tsconfig.build.json",`
	`3`	`+ "include": ["src", "tests"],`
	`4`	`+ "exclude": ["tests/test-project", "tests/test-tslint-rules-directory"]`
`8`	`5`	`}`