Changes as suggested in review

Update ci workflow with currently supported NodeJS version
Add variable to catch statement
Add appropriate eslint-disable directive
Update ESLint config
Add Node 20 to CI script
Updated CI script versions

Author: droberts-ctrlo

.github/workflows/ci.yaml

@@ -13,12 +13,12 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x, 14.x, 16.x, 17.x]
+        node-version: [20.x, 22.x, 24.x, 25.x]
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v6
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2
+      uses: actions/setup-node@v6
       with:
         node-version: ${{ matrix.node-version }}
         cache: 'yarn'

eslint.config.mjs

@@ -2,7 +2,8 @@ import globals from "globals";
 import { defineConfig } from "eslint/config";
 
 export default defineConfig([
-	{ files: ["**/*.{js,mjs,cjs}"], languageOptions: { globals: {...globals.browser, ...globals.node, ...globals.worker, ...globals.jest} } },
+	{ files: ["**/*.{js,mjs,cjs}"], languageOptions: { globals: {...globals.browser, ...globals.node, ...globals.worker} } },
+	{ files: ["test/**/*.{js,mjs,cjs}"], languageOptions: { globals: globals.jest } },
 	{
 		"rules": {
 			"indent": [
@@ -25,7 +26,8 @@ export default defineConfig([
 			"no-unused-vars": [
 				2,
 				{
-					"args": "none"
+					"args": "none",
+					"caughtErrors": "none"
 				}
 			],
 			"semi": [

src/js.js

@@ -86,7 +86,7 @@ let custom_inspect_symbol;
 if (typeof process !== "undefined") {
 	try { /* for node.js */
 		custom_inspect_symbol = require('util').inspect.custom;
-	} catch {}
+	} catch (e) {}
 }
 
 const global_env = (function(Object) {