Add eslint validation (#33)

.eslintrc

@@ -0,0 +1,41 @@
+{
+    "env": {
+        "browser": true,
+        "es2021": true
+    },
+    "extends": [
+        "eslint:recommended",
+        "plugin:@typescript-eslint/recommended",
+        "plugin:security/recommended"
+    ],
+    "plugins": [
+        "security"
+    ],
+    "rules": {
+        "quotes": ["warn", "double"],                               // Enabled for auto fixing
+        "prefer-const": "off",
+        "prefer-spread": "off",
+        "no-var": "off",
+        "no-extra-boolean-cast": "off",
+        "prefer-rest-params": "off",
+        "no-case-declarations": "off",
+        "no-prototype-builtins": "off",
+        "no-useless-escape": "off",                                 // Suppressing Error -- need to Review Later
+        "no-trailing-spaces": [ "warn", { "skipBlankLines": true }],// Enabled for auto fixing
+        "no-const-assign": "error",
+        "comma-dangle": [ "error", "never" ],                       // Enabled for auto fixing
+        "security/detect-object-injection": "off",                  // Suppress Warning -- need to Review Later
+        "@typescript-eslint/ban-types": "off",
+        "@typescript-eslint/no-unused-vars": [ "warn", { "vars": "all", "args": "none", "argsIgnorePattern": "^_", "ignoreRestSiblings": true } ],
+        "@typescript-eslint/triple-slash-reference": "off",
+        "@typescript-eslint/no-inferrable-types": "off",
+        "@typescript-eslint/no-this-alias": "off",
+        "@typescript-eslint/no-explicit-any": "off",
+        "@typescript-eslint/no-empty-function": "off",
+        "@typescript-eslint/no-empty-interface": "off",
+        "@typescript-eslint/no-var-requires": "off",
+        "@typescript-eslint/explicit-module-boundary-types": "off",
+        "@typescript-eslint/no-extra-semi": "error",                // Enabled for auto fixing
+        "@typescript-eslint/no-non-null-assertion": "error"
+    }
+}

.github/workflows/linux.yml

@@ -20,6 +20,7 @@ jobs:
       - run: npm ci
       - run: npm run build:ts
       - run: npm run build
+      - run: if [[ ${{ matrix.node-version }} == 14 ]]; then npm run lint; fi;
       - run: npm test
       - run: npm run package
       - uses: actions/upload-artifact@v2

package.json

@@ -21,6 +21,7 @@
     "install": "node-pre-gyp install --fallback-to-build",
     "build:ts": "tsc --project ./tsconfig.json",
     "build": "node-gyp configure && node-gyp rebuild && npm run build:ts",
+    "lint": "eslint ./lib --fix",
     "test": "mocha --recursive --expose-gc test/EndToEnd.js",
     "test:native": "run-s test:renamebinding test:renametestbinding build test:runtests test:renameoriginalbinding",
     "test:runtests": "node ./test_native/test.js",
@@ -45,7 +46,15 @@
   },
   "devDependencies": {
     "@types/node": "^11.15.2",
+    "@typescript-eslint/eslint-plugin": "^5.11.0",
+    "@typescript-eslint/parser": "^5.11.0",
     "copyfiles": "^2.1.1",
+    "eslint": "^7.29.0",
+    "eslint-config-standard": "^16.0.3",
+    "eslint-plugin-import": "^2.23.4",
+    "eslint-plugin-node": "^11.1.0",
+    "eslint-plugin-promise": "^5.1.0",
+    "eslint-plugin-security": "^1.4.0",
     "mocha": "^5.2.0",
     "npm-run-all": "^4.1.5",
     "rename-files": "0.0.2",