Switch to eslint

code-asher · code-asher · commit beba925d75db · 2021-04-26T16:11:55.000-05:00
Fix the Google Cloud logger types as well.
diff --git a/.editorconfig b/.editorconfig
@@ -0,0 +1,6 @@
+root = true
+
+[*]
+indent_style = space
+trim_trailing_whitespace = true
+indent_size = 2
diff --git a/.eslintrc.yaml b/.eslintrc.yaml
@@ -0,0 +1,27 @@
+parser: "@typescript-eslint/parser"
+env:
+  browser: true
+  es6: true # Map, etc.
+  jest: true
+  node: true
+
+parserOptions:
+  ecmaVersion: 2018
+  sourceType: module
+
+extends:
+  - eslint:recommended
+  - plugin:@typescript-eslint/recommended
+  - plugin:import/recommended
+  - plugin:import/typescript
+  - plugin:prettier/recommended
+  # Prettier should always be last
+  # Removes eslint rules that conflict with prettier.
+  - prettier
+
+rules:
+  # For overloads.
+  no-dupe-class-members: off
+  eqeqeq: error
+  import/order:
+    [error, { alphabetize: { order: "asc" }, groups: [["builtin", "external", "internal"], "parent", "sibling"] }]
diff --git a/.prettierrc.yaml b/.prettierrc.yaml
@@ -0,0 +1,4 @@
+printWidth: 120
+semi: false
+trailingComma: all
+arrowParens: always
diff --git a/.stylelintrc.yaml b/.stylelintrc.yaml
@@ -0,0 +1,2 @@
+extends:
+  - stylelint-config-recommended
diff --git a/package.json b/package.json
@@ -1,34 +1,42 @@
 {
-	"name": "@coder/logger",
-	"description": "Beautiful logging inspired by https://github.com/uber-go/zap.",
-	"scripts": {
-		"prepare": "tsc",
-		"test": "jest"
-	},
-	"version": "1.1.16",
-	"main": "out/index.js",
-	"types": "out/index.d.ts",
-	"author": "Coder",
-	"license": "MIT",
-	"peerDependencies": {
-		"@google-cloud/logging": "^4.5.2"
-	},
-	"devDependencies": {
-		"@types/jest": "^24.0.16",
-		"@types/node": "^12.6.8",
-		"jest": "^24.8.0",
-		"ts-jest": "^24.0.2",
-		"tslint": "^5.18.0",
-		"typescript": "^3.5.3"
-	},
-	"jest": {
-		"moduleFileExtensions": [
-			"js",
-			"ts"
-		],
-		"transform": {
-			"^.+\\.tsx?$": "ts-jest"
-		},
-		"testRegex": ".*\\.test\\.ts"
-	}
+  "name": "@coder/logger",
+  "description": "Beautiful logging inspired by https://github.com/uber-go/zap.",
+  "scripts": {
+    "lint": "eslint src/**",
+    "prepare": "tsc",
+    "test": "jest"
+  },
+  "version": "1.1.16",
+  "main": "out/index.js",
+  "types": "out/index.d.ts",
+  "author": "Coder",
+  "license": "MIT",
+  "peerDependencies": {
+    "@google-cloud/logging": "^9.2.1"
+  },
+  "devDependencies": {
+    "@google-cloud/logging": "^9.2.1",
+    "@types/jest": "^24.0.16",
+    "@types/node": "^12.6.8",
+    "@typescript-eslint/eslint-plugin": "^4.22.0",
+    "@typescript-eslint/parser": "^4.22.0",
+    "eslint": "^7.25.0",
+    "eslint-config-prettier": "^8.3.0",
+    "eslint-plugin-import": "^2.22.1",
+    "eslint-plugin-prettier": "^3.4.0",
+    "jest": "^24.8.0",
+    "prettier": "^2.2.1",
+    "ts-jest": "^24.0.2",
+    "typescript": "^3.5.3"
+  },
+  "jest": {
+    "moduleFileExtensions": [
+      "js",
+      "ts"
+    ],
+    "transform": {
+      "^.+\\.tsx?$": "ts-jest"
+    },
+    "testRegex": ".*\\.test\\.ts"
+  }
 }
diff --git a/src/extender.test.ts b/src/extender.test.ts
@@ -1,12 +1,12 @@
-import { field, logger } from "./logger";
-import { createStackdriverExtender } from "./extender";
+import { createStackdriverExtender } from "./extender"
+import { field, logger } from "./logger"
 
 describe("Extender", () => {
-	it("should add stackdriver extender", () => {
-		logger.extend(createStackdriverExtender("coder-dev-1", "logging-package-tests"));
-	});
+  it("should add stackdriver extender", () => {
+    logger.extend(createStackdriverExtender("coder-dev-1", "logging-package-tests"))
+  })
 
-	it("should log", async () => {
-		logger.debug("Bananas!", field("frog", { hi: "wow" }));
-	});
-});
+  it("should log", async () => {
+    logger.debug("Bananas!", field("frog", { hi: "wow" }))
+  })
+})
diff --git a/src/extender.ts b/src/extender.ts
@@ -1,64 +1,53 @@
-// @ts-ignore
-import * as gcl from "@google-cloud/logging";
-import { Extender, logger, field } from "./logger";
+import * as gcl from "@google-cloud/logging"
+import { Extender, logger, field } from "./logger"
 
 export const createStackdriverExtender = (projectId: string, logId: string): Extender => {
-	enum GcpLogSeverity {
-		DEFAULT = 0,
-		DEBUG = 100,
-		INFO = 200,
-		NOTICE = 300,
-		WARNING = 400,
-		ERROR = 500,
-		CRITICAL = 600,
-		ALERT = 700,
-		EMERGENCY = 800,
-	}
+  const logging = new gcl.Logging({
+    autoRetry: true,
+    projectId,
+  })
 
-	const logging = new gcl.Logging({
-		autoRetry: true,
-		projectId,
-	});
+  const log = logging.log(logId)
+  const convertSeverity = (
+    severity: "trace" | "info" | "warn" | "debug" | "error",
+  ): gcl.protos.google.logging.type.LogSeverity => {
+    switch (severity) {
+      case "trace":
+      case "debug":
+        return gcl.protos.google.logging.type.LogSeverity.DEBUG
+      case "info":
+        return gcl.protos.google.logging.type.LogSeverity.INFO
+      case "error":
+        return gcl.protos.google.logging.type.LogSeverity.ERROR
+      case "warn":
+        return gcl.protos.google.logging.type.LogSeverity.WARNING
+    }
+  }
 
-	const log = logging.log(logId);
-	const convertSeverity = (severity: "trace" | "info" | "warn" | "debug" | "error"): GcpLogSeverity => {
-		switch (severity) {
-			case "trace":
-			case "debug":
-				return GcpLogSeverity.DEBUG;
-			case "info":
-				return GcpLogSeverity.INFO;
-			case "error":
-				return GcpLogSeverity.ERROR;
-			case "warn":
-				return GcpLogSeverity.WARNING;
-		}
-	};
+  return (options): void => {
+    const severity = convertSeverity(options.type)
+    const metadata: { [id: string]: string } = {}
+    if (options.fields) {
+      options.fields.forEach((f) => {
+        if (!f) {
+          return
+        }
+        metadata[f.identifier] = f.value
+      })
+    }
 
-	return (options): void => {
-		const severity = convertSeverity(options.type);
-		// tslint:disable-next-line:no-any
-		const metadata = {} as any;
-		if (options.fields) {
-			options.fields.forEach((f) => {
-				if (!f) {
-					return;
-				}
-				metadata[f.identifier] = f.value;
-			});
-		}
+    const entry = log.entry(
+      {
+        severity: severity,
+      },
+      {
+        ...metadata,
+        message: options.message,
+      },
+    )
 
-		const entry = log.entry({
-			// tslint:disable-next-line:no-any
-			severity: severity as any,
-		}, {
-			...metadata,
-			message: options.message,
-		});
-
-		log.write(entry).catch((ex: Error) => {
-			logger.named("GCP").error("Failed to log", field("error", ex));
-		});
-	};
-
-};
+    log.write(entry).catch((ex: Error) => {
+      logger.named("GCP").error("Failed to log", field("error", ex))
+    })
+  }
+}
diff --git a/src/index.ts b/src/index.ts
@@ -1 +1 @@
-export * from "./logger";
+export * from "./logger"
diff --git a/src/logger.test.ts b/src/logger.test.ts
@@ -1,16 +1,16 @@
-import { field, logger, BrowserFormatter, Time } from "./logger";
+import { field, logger, BrowserFormatter, Time } from "./logger"
 
 describe("Logger", () => {
-	it("should use server formatter", () => {
-		logger.info("test", field("key", "value"), field("time", new Time(100, Date.now())));
-		logger.named("name").debug("test name");
-		logger.named("another name").warn("another test name");
-	});
+  it("should use server formatter", () => {
+    logger.info("test", field("key", "value"), field("time", new Time(100, Date.now())))
+    logger.named("name").debug("test name")
+    logger.named("another name").warn("another test name")
+  })
 
-	it("should use browser formatter", () => {
-		logger.formatter = new BrowserFormatter();
-		logger.info("test", field("key", "value"), field("time", new Time(100, Date.now())));
-		logger.named("name").debug("test name");
-		logger.named("another name").warn("another test name");
-	});
-});
+  it("should use browser formatter", () => {
+    logger.formatter = new BrowserFormatter()
+    logger.info("test", field("key", "value"), field("time", new Time(100, Date.now())))
+    logger.named("name").debug("test name")
+    logger.named("another name").warn("another test name")
+  })
+})
diff --git a/src/logger.ts b/src/logger.ts
diff --git a/tslint.json b/tslint.json
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+extends:`
	`2`	`+ - stylelint-config-recommended`
Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-export * from "./logger";`
	`1`	`+export * from "./logger"`