Upgrade typescript-eslint and prettier (#92)

Author: ota-meshi

.eslintrc.yml

@@ -6,7 +6,13 @@ globals: {
 }
 overrides:
   - files: "**/*.ts"
+    plugins:
+      - "@typescript-eslint"
     parser: "@typescript-eslint/parser"
+    rules:
+      "@typescript-eslint/consistent-type-imports": error
+      no-duplicate-imports: off
+      "@typescript-eslint/no-duplicate-imports": error
   - files: "typings/**"
     rules:
       node/no-missing-import:

package.json

@@ -28,6 +28,7 @@
     "@types/lodash": "^4.14.120",
     "@types/mocha": "^5.2.4",
     "@types/node": "^10.12.21",
+    "@typescript-eslint/eslint-plugin": "^4.9.1",
     "@typescript-eslint/parser": "^4.7.0",
     "babel-eslint": "^10.0.1",
     "chokidar": "^2.0.4",
@@ -41,6 +42,7 @@
     "npm-run-all": "^4.1.5",
     "nyc": "^14.0.0",
     "opener": "^1.5.1",
+    "prettier": "^2.2.1",
     "rimraf": "^2.6.3",
     "rollup": "^1.1.2",
     "rollup-plugin-node-resolve": "^4.0.0",

src/ast/errors.ts

@@ -3,7 +3,7 @@
  * @copyright 2017 Toru Nagashima. All rights reserved.
  * See LICENSE file in root directory for full license.
  */
-import { Location } from "./locations"
+import type { Location } from "./locations"
 
 /**
  * Check whether the given value has acorn style location information.

src/ast/nodes.ts

@@ -3,10 +3,10 @@
  * @copyright 2017 Toru Nagashima. All rights reserved.
  * See LICENSE file in root directory for full license.
  */
-import { ScopeManager } from "eslint-scope"
-import { ParseError } from "./errors"
-import { HasLocation } from "./locations"
-import { Token } from "./tokens"
+import type { ScopeManager } from "eslint-scope"
+import type { ParseError } from "./errors"
+import type { HasLocation } from "./locations"
+import type { Token } from "./tokens"
 
 //------------------------------------------------------------------------------
 // Common

src/ast/tokens.ts

@@ -3,7 +3,7 @@
  * @copyright 2017 Toru Nagashima. All rights reserved.
  * See LICENSE file in root directory for full license.
  */
-import { HasLocation } from "./locations"
+import type { HasLocation } from "./locations"
 
 /**
  * Tokens.

src/ast/traverse.ts

@@ -3,8 +3,9 @@
  * @copyright 2017 Toru Nagashima. All rights reserved.
  * See LICENSE file in root directory for full license.
  */
-import Evk, { VisitorKeys } from "eslint-visitor-keys"
-import { Node } from "./nodes"
+import type { VisitorKeys } from "eslint-visitor-keys"
+import Evk from "eslint-visitor-keys"
+import type { Node } from "./nodes"
 
 //------------------------------------------------------------------------------
 // Helpers

src/common/fix-locations.ts

@@ -1,10 +1,6 @@
-import {
-    ESLintExtendedProgram,
-    LocationRange,
-    Node,
-    traverseNodes,
-} from "../ast"
-import { LocationCalculator } from "./location-calculator"
+import type { ESLintExtendedProgram, LocationRange, Node } from "../ast"
+import { traverseNodes } from "../ast"
+import type { LocationCalculator } from "./location-calculator"
 
 /**
  * Do post-process of parsing an expression.

src/common/location-calculator.ts

@@ -4,7 +4,7 @@
  * See LICENSE file in root directory for full license.
  */
 import sortedLastIndex from "lodash/sortedLastIndex"
-import { HasLocation, Location, ParseError } from "../ast"
+import type { HasLocation, Location, ParseError } from "../ast"
 
 /**
  * Location calculators.

src/html/intermediate-tokenizer.ts

@@ -5,16 +5,16 @@
  */
 import assert from "assert"
 import last from "lodash/last"
-import {
+import type {
     ErrorCode,
     HasLocation,
     Namespace,
-    ParseError,
     Token,
     VAttribute,
 } from "../ast"
+import { ParseError } from "../ast"
 import { debug } from "../common/debug"
-import { Tokenizer, TokenizerState, TokenType } from "./tokenizer"
+import type { Tokenizer, TokenizerState, TokenType } from "./tokenizer"
 
 const DUMMY_PARENT: any = Object.freeze({})
 

src/html/parser.ts

@@ -6,18 +6,17 @@
 import assert from "assert"
 import last from "lodash/last"
 import findLastIndex from "lodash/findLastIndex"
-import {
+import type {
     ErrorCode,
     HasLocation,
     Namespace,
-    NS,
-    ParseError,
     Token,
     VAttribute,
     VDocumentFragment,
     VElement,
     VExpressionContainer,
 } from "../ast"
+import { NS, ParseError } from "../ast"
 import { debug } from "../common/debug"
 import { LocationCalculator } from "../common/location-calculator"
 import {
@@ -37,16 +36,17 @@ import {
     HTML_VOID_ELEMENT_TAGS,
     SVG_ELEMENT_NAME_MAP,
 } from "./util/tag-names"
-import {
+import type {
     IntermediateToken,
-    IntermediateTokenizer,
     EndTag,
     Mustache,
     StartTag,
     Text,
 } from "./intermediate-tokenizer"
-import { Tokenizer } from "./tokenizer"
-import { isSFCFile, ParserOptions } from "../common/parser-options"
+import { IntermediateTokenizer } from "./intermediate-tokenizer"
+import type { Tokenizer } from "./tokenizer"
+import type { ParserOptions } from "../common/parser-options"
+import { isSFCFile } from "../common/parser-options"
 
 const DIRECTIVE_NAME = /^(?:v-|[.:@#]).*[^.:@#]$/u
 const DT_DD = /^d[dt]$/u
@@ -83,7 +83,7 @@ function isHTMLIntegrationPoint(element: VElement): boolean {
         return (
             element.name === "annotation-xml" &&
             element.startTag.attributes.some(
-                a =>
+                (a) =>
                     a.directive === false &&
                     a.key.name === "encoding" &&
                     a.value != null &&
@@ -358,7 +358,7 @@ export class Parser {
         }
 
         if (name === "template") {
-            const xmlns = token.attributes.find(a => a.key.name === "xmlns")
+            const xmlns = token.attributes.find((a) => a.key.name === "xmlns")
             const value = xmlns && xmlns.value && xmlns.value.value
 
             if (value === NS.HTML || value === NS.MathML || value === NS.SVG) {
@@ -462,7 +462,7 @@ export class Parser {
         }
         const hasVPre =
             !this.isInVPreElement &&
-            token.attributes.some(a => a.key.name === "v-pre")
+            token.attributes.some((a) => a.key.name === "v-pre")
 
         // Disable expression if v-pre
         if (hasVPre) {
@@ -520,7 +520,7 @@ export class Parser {
         if (namespace === NS.HTML) {
             if (element.parent.type === "VDocumentFragment") {
                 const langAttr = element.startTag.attributes.find(
-                    a => !a.directive && a.key.name === "lang",
+                    (a) => !a.directive && a.key.name === "lang",
                 ) as VAttribute | undefined
                 const lang = langAttr?.value?.value
 
@@ -565,7 +565,7 @@ export class Parser {
 
         const i = findLastIndex(
             this.elementStack,
-            el => el.name.toLowerCase() === token.name,
+            (el) => el.name.toLowerCase() === token.name,
         )
         if (i === -1) {
             this.reportParseError(token, "x-invalid-end-tag")