feat: replace tslint with eslint

See https://github.com/typescript-eslint/typescript-eslint

Author: raymondfeng

.eslintignore

@@ -3,3 +3,4 @@ dist/
 coverage/
 api-docs/
 /sandbox
+**/*.d.ts

.travis.yml

@@ -27,7 +27,7 @@ matrix:
       env: TASK=code-lint
       # Running Code Linter -- Requires @loopback/build so it's bootstrapped
       script:
-        - lerna bootstrap --scope @loopback/build --scope @loopback/tslint-config --scope @loopback/eslint-config
+        - lerna bootstrap --scope @loopback/build --scope @loopback/eslint-config
         - npm run lint
     - node_js: "8"
       os: linux

.vscode/settings.json

@@ -26,7 +26,15 @@
   },
   "files.insertFinalNewline": true,
   "files.trimTrailingWhitespace": true,
-  "tslint.ignoreDefinitionFiles": true,
-  "tslint.nodePath": "./packages/build/node_modules",
-  "typescript.tsdk": "./packages/build/node_modules/typescript/lib"
+  "typescript.tsdk": "./packages/build/node_modules/typescript/lib",
+  "eslint.autoFixOnSave": true,
+  "eslint.run": "onSave",
+  "eslint.nodePath": "./packages/build/node_modules",
+  "eslint.validate": [
+    "javascript",
+    {
+        "language": "typescript",
+        "autoFix": true
+    }
+  ]
 }

.vscode/tasks.json

@@ -34,7 +34,7 @@
       "args": [
         "test"
       ],
-      "problemMatcher": ["$tsc", "$tslint5"]
+      "problemMatcher": ["$tsc", "$eslint-stylish", "$eslint-compact"]
     },
     {
       "label": "Lint all packages",
@@ -47,10 +47,11 @@
         "lint"
       ],
       "problemMatcher": [
-        "$tslint5",
+        "$eslint-stylish",
+        "$eslint-compact",
         {
           // compilation errors - relative paths
-          "owner": "tslint",
+          "owner": "eslint",
           "fileLocation": [
             "relative",
             "${workspaceRoot}"
@@ -66,7 +67,7 @@
         },
         {
           // compilation errors - absolute paths
-          "owner": "tslint",
+          "owner": "eslint",
           "fileLocation": "absolute",
           "severity": "error",
           "pattern": {

benchmark/src/__tests__/benchmark.integration.ts

@@ -17,7 +17,7 @@ const DUMMY_STATS: EndpointStats = {
 
 describe('Benchmark (SLOW)', function() {
   // Unfortunately, the todo app requires one second to start
-  // tslint:disable-next-line:no-invalid-this
+  // eslint-disable-next-line no-invalid-this
   this.timeout(5000);
   it('works', async () => {
     const bench = new Benchmark();
@@ -37,7 +37,7 @@ describe('Benchmark (SLOW)', function() {
     async execute(
       title: string,
       urlPath: string,
-      // tslint:disable-next-line:no-any
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       options?: any,
     ): Promise<EndpointStats> {
       if (!options) options = {};

benchmark/src/rest-routing/routing-table.ts

@@ -90,7 +90,7 @@ function givenRouter(router: RestRouter, spec: OpenApiSpec, count: number) {
         // Make it not found
         group = 'groupX';
       }
-      // tslint:disable-next-line:no-any
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       const request: any = {
         method: 'get',
         path: `/my/hello/${group}/version_${i}`,

bin/build-docs-site.sh

@@ -22,7 +22,7 @@ REPO_ROOT=$DIR/..
 pushd $REPO_ROOT >/dev/null
 
 # Update apidocs
-lerna bootstrap --scope @loopback/tsdocs
+lerna bootstrap
 lerna run --scope @loopback/docs prepack
 
 # Clean up sandbox/loopback.io directory
@@ -31,8 +31,8 @@ rm -rf sandbox/loopback.io/
 # Shadow clone the `strongloop/loopback.io` github repo
 git clone --depth 1 https://github.com/strongloop/loopback.io.git sandbox/loopback.io
 
-# Bootstrap the `loopback.io` package
-lerna bootstrap --scope loopback.io-workflow-scripts --scope @loopback/docs
+# Add loopback.io-workflow-scripts with @loopback/docs linked
+lerna bootstrap --scope @loopback/docs --scope loopback.io-workflow-scripts
 
 pushd $REPO_ROOT/sandbox/loopback.io/ >/dev/null
 

bin/run-lerna.js

@@ -19,7 +19,7 @@ async function run(argv, options) {
   const rootPath = project.rootPath;
 
   process.env.LERNA_ROOT_PATH = rootPath;
-  let args = argv.slice(2);
+  const args = argv.slice(2);
 
   return build.runCLI('lerna/cli', args, options);
 }

bin/sync-dev-deps.js

@@ -25,13 +25,27 @@ async function syncDevDeps() {
   const buildDeps = require(path.join(rootPath, 'packages/build/package.json'))
     .dependencies;
 
-  const masterDeps = {
-    typescript: buildDeps.typescript,
-    tslint: buildDeps.tslint,
-  };
+  const deps = [
+    'typescript',
+    'eslint',
+    '@typescript-eslint/eslint-plugin',
+    '@typescript-eslint/parser',
+    'eslint-plugin-eslint-plugin',
+  ];
+  const masterDeps = {};
+  for (const d of deps) {
+    if (buildDeps[d] == null) {
+      console.error(
+        'Dependency %s is missing in packages/build/package.json',
+        d,
+      );
+    }
+    masterDeps[d] = buildDeps[d];
+  }
 
-  // Update typescript & tslint dependencies in individual packages
+  // Update typescript & eslint dependencies in individual packages
   for (const pkg of packages) {
+    if (pkg.name === '@loopback/build') continue;
     const pkgFile = pkg.manifestLocation;
     updatePackageJson(pkgFile, masterDeps);
   }
@@ -48,9 +62,20 @@ async function syncDevDeps() {
  */
 function updatePackageJson(pkgFile, masterDeps) {
   const data = readJsonFile(pkgFile);
+  const isExample = data.name.startsWith('@loopback/example-');
+  const isRoot = data.name === 'loopback-next';
+
   let modified = false;
+  if (isExample && data.devDependencies && data.devDependencies.tslint) {
+    delete data.devDependencies.tslint;
+    modified = true;
+  }
   for (const dep in masterDeps) {
-    if (data.devDependencies && dep in data.devDependencies) {
+    if (
+      data.devDependencies &&
+      // Force update for examples and loopback-next
+      (isExample || isRoot || dep in data.devDependencies)
+    ) {
       modified = modified || data.devDependencies[dep] !== masterDeps[dep];
       data.devDependencies[dep] = masterDeps[dep];
     }

docs/site/Application-generator.md

@@ -22,7 +22,7 @@ lb4 [app] [options] [<name>]
 
 `--outDir` : Project root directory for the application.
 
-`--tslint` : Add TSLint to LoopBack4 application project.
+`--eslint` : Add ESLint to LoopBack4 application project.
 
 `--prettier` : Add Prettier to LoopBack4 application project.
 
@@ -65,7 +65,7 @@ The tool will prompt you for:
   help format, test, and build a LoopBack4 application. Defaults to `true` for
   all of the modules. The prompted modules are:
 
-  - [`tslint`](https://www.npmjs.com/package/tslint)
+  - [`eslint`](https://www.npmjs.com/package/eslint)
   - [`prettier`](https://www.npmjs.com/package/prettier)
   - [`mocha`](https://www.npmjs.com/package/mocha)
   - [`@loopback/build`](https://www.npmjs.com/package/@loopback/build)