Update dependencies to latest versions, set minimum support to node v…

…18, fix linter warnings and err ors.
slackapi · Aug 25, 2023 · 9fdbacc · 9fdbacc
1 parent 6afaae0
commit 9fdbacc
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 25 deletions.
diff --git a/packages/logger/package.json b/packages/logger/package.json
@@ -14,8 +14,8 @@
     "dist/**/*"
   ],
   "engines": {
-    "node": ">= 12.13.0",
-    "npm": ">= 6.12.0"
+    "node": ">= 18",
+    "npm": ">= 8.6.0"
   },
   "repository": "slackapi/node-slack-sdk",
   "homepage": "https://slack.dev/node-slack-sdk",
@@ -30,31 +30,34 @@
     "build": "npm run build:clean && tsc",
     "build:clean": "shx rm -rf ./dist ./coverage ./.nyc_output",
     "lint": "eslint --ext .ts src",
-    "test": "npm run lint && npm run build && nyc mocha --config .mocharc.json src/*.spec.js",
+    "mocha": "mocha --config .mocharc.json src/*.spec.js",
+    "test:unit": "npm run build && npm run mocha",
+    "test": "npm run lint && npm run test:unit",
+    "coverage": "npm run build && nyc --reporter=text-summary npm run mocha",
     "ref-docs:model": "api-extractor run"
   },
   "dependencies": {
-    "@types/node": ">=12.0.0"
+    "@types/node": ">=18.0.0"
   },
   "devDependencies": {
-    "@microsoft/api-extractor": "^7.3.4",
-    "@typescript-eslint/eslint-plugin": "^4.4.1",
-    "@typescript-eslint/parser": "^4.4.0",
-    "@types/chai": "^4.1.7",
-    "@types/mocha": "^5.2.6",
-    "chai": "^4.2.0",
-    "eslint": "^7.32.0",
-    "eslint-config-airbnb-base": "^14.2.1",
-    "eslint-config-airbnb-typescript": "^12.3.1",
-    "eslint-plugin-import": "^2.22.1",
-    "eslint-plugin-jsdoc": "^30.6.1",
+    "@microsoft/api-extractor": "^7.36.4",
+    "@typescript-eslint/eslint-plugin": "^6.4.1",
+    "@typescript-eslint/parser": "^6.4.0",
+    "@types/chai": "^4.3.5",
+    "@types/mocha": "^10.0.1",
+    "chai": "^4.3.8",
+    "eslint": "^8.47.0",
+    "eslint-config-airbnb-base": "^15.0.0",
+    "eslint-config-airbnb-typescript": "^17.1.0",
+    "eslint-plugin-import": "^2.28.1",
+    "eslint-plugin-jsdoc": "^46.5.0",
     "eslint-plugin-node": "^11.1.0",
-    "mocha": "^9.1.0",
-    "nyc": "^14.1.1",
+    "mocha": "^10.2.0",
+    "nyc": "^15.1.0",
     "shx": "^0.3.2",
     "ts-node": "^8.2.0",
-    "sinon": "^7.3.2",
-    "source-map-support": "^0.5.12",
+    "sinon": "^15.2.0",
+    "source-map-support": "^0.5.21",
     "typescript": "^4.1.0"
   }
 }
diff --git a/packages/logger/src/index.ts b/packages/logger/src/index.ts
@@ -16,36 +16,35 @@ export enum LogLevel {
 export interface Logger {
   /**
    * Output debug message
-   *
    * @param msg any data to log
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   debug(...msg: any[]): void;
 
   /**
    * Output info message
-   *
    * @param msg any data to log
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   info(...msg: any[]): void;
 
   /**
    * Output warn message
-   *
    * @param msg any data to log
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   warn(...msg: any[]): void;
 
   /**
    * Output error message
-   *
    * @param msg any data to log
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   error(...msg: any[]): void;
 
   /**
    * This disables all logging below the given level, so that after a log.setLevel("warn") call log.warn("something")
    * or log.error("something") will output messages, but log.info("something") will not.
-   *
    * @param level as a string, like 'error' (case-insensitive)
    */
   setLevel(level: LogLevel): void;
@@ -58,7 +57,6 @@ export interface Logger {
   /**
    * This allows the instance to be named so that they can easily be filtered when many loggers are sending output
    * to the same destination.
-   *
    * @param name as a string, will be output with every log after the level
    */
   setName(name: string): void;
@@ -115,6 +113,7 @@ export class ConsoleLogger implements Logger {
   /**
    * Log a debug message
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   public debug(...msg: any[]): void {
     if (ConsoleLogger.isMoreOrEqualSevere(LogLevel.DEBUG, this.level)) {
       console.debug(ConsoleLogger.labels.get(LogLevel.DEBUG), this.name, ...msg);
@@ -124,6 +123,7 @@ export class ConsoleLogger implements Logger {
   /**
    * Log an info message
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   public info(...msg: any[]): void {
     if (ConsoleLogger.isMoreOrEqualSevere(LogLevel.INFO, this.level)) {
       console.info(ConsoleLogger.labels.get(LogLevel.INFO), this.name, ...msg);
@@ -133,6 +133,7 @@ export class ConsoleLogger implements Logger {
   /**
    * Log a warning message
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   public warn(...msg: any[]): void {
     if (ConsoleLogger.isMoreOrEqualSevere(LogLevel.WARN, this.level)) {
       console.warn(ConsoleLogger.labels.get(LogLevel.WARN), this.name, ...msg);
@@ -142,6 +143,7 @@ export class ConsoleLogger implements Logger {
   /**
    * Log an error message
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   public error(...msg: any[]): void {
     if (ConsoleLogger.isMoreOrEqualSevere(LogLevel.ERROR, this.level)) {
       console.error(ConsoleLogger.labels.get(LogLevel.ERROR), this.name, ...msg);