Merge pull request #167 from msgpack/importsNotUsedAsValues

importsNotUsedAsValues: error; the most strict way to control imports

Author: gfx

src/Encoder.ts

@@ -2,7 +2,7 @@ import { utf8EncodeJs, utf8Count, TEXT_ENCODER_THRESHOLD, utf8EncodeTE } from ".
 import { ExtensionCodec, ExtensionCodecType } from "./ExtensionCodec";
 import { setInt64, setUint64 } from "./utils/int";
 import { ensureUint8Array } from "./utils/typedArrays";
-import { ExtData } from "./ExtData";
+import type { ExtData } from "./ExtData";
 
 export const DEFAULT_MAX_DEPTH = 100;
 export const DEFAULT_INITIAL_BUFFER_SIZE = 2048;

src/decode.ts

@@ -1,6 +1,6 @@
-import { ExtensionCodecType } from "./ExtensionCodec";
 import { Decoder } from "./Decoder";
-import { ContextOf, SplitUndefined } from "./context";
+import type { ExtensionCodecType } from "./ExtensionCodec";
+import type { ContextOf, SplitUndefined } from "./context";
 
 export type DecodeOptions<ContextType = undefined> = Readonly<
   Partial<{

src/encode.ts

@@ -1,6 +1,6 @@
-import { ExtensionCodecType } from "./ExtensionCodec";
 import { Encoder } from "./Encoder";
-import { ContextOf, SplitUndefined } from "./context";
+import type { ExtensionCodecType } from "./ExtensionCodec";
+import type { ContextOf, SplitUndefined } from "./context";
 
 export type EncodeOptions<ContextType = undefined> = Partial<
   Readonly<{

test/decode-max-length.test.ts

@@ -1,6 +1,6 @@
 import assert from "assert";
 import { encode, decode, decodeAsync } from "../src";
-import { DecodeOptions } from "../src/decode";
+import type { DecodeOptions } from "../src/decode";
 
 describe("decode with max${Type}Length specified", () => {
   async function* createStream<T>(input: T) {

tsconfig.json

@@ -39,6 +39,7 @@
     "noFallthroughCasesInSwitch": true,    /* Report errors for fallthrough cases in switch statement. */
     "noUncheckedIndexedAccess": true,
     "noPropertyAccessFromIndexSignature": true,
+    "importsNotUsedAsValues": "error",
 
     /* Module Resolution Options */
     "moduleResolution": "node",            /* Specify module resolution strategy: 'node' (Node.js) or 'classic' (TypeScript pre-1.6). */