refactor: #158 configure no-undef eslint rule (#176)

* removed tslint

* add eslint

* add default config

* setup default config and npm run command

* add command to node.js.yml

* set rules to support current codebase

* remove unused tslint.json

* include sample files in eslint command

* remove rule no-explicit-any

* specify type for timer

* remove any from filterKeys

* explictly declare that error can be Error, String but still accept any from the end users

* specify types in the sync worker

* improve the IndexableError type

* explain why any is allowed in CustomData

* remove no-this-alias from eslint config

* convert to arrow function to preserve this reference

* use arrow functions to remove the need of a this alias

* define httpoptions object instead of using this alias

* add examples to prettier command

* add prettier job to node js ci job

* script fix

* setup no-unused-vars lint rule

* remove unused var

* fixed lib/raygun.batch.ts

* fix lib/raygun.offline.ts

* cleanup lib/raygun.sync.transport.ts

* fix lib/raygun.transport.ts

* fix lib/raygun.ts

* fix @typescript-eslint/no-unused-vars

* fixs in test files

* update package lock in sample

* cleanup eslint config

* config files

* run npm run prettier

* chore: document why no-var-requires ignore is necessary

update comment

* fix: #158 configure no-undef eslint rule

.github/workflows/node.js.yml

@@ -39,6 +39,7 @@ jobs:
     steps:
       - name: Checkout
         uses: actions/checkout@v3
+        with:
           # Make sure the actual branch is checked out when running on pull requests
           ref: ${{ github.head_ref }}
           # Make sure the value of GITHUB_TOKEN will not be persisted in repo's config

eslint.config.mjs

@@ -7,6 +7,17 @@ export default tseslint.config(
     eslint.configs.recommended,
     ...tseslint.configs.recommended,
     {
+        languageOptions: {
+            // Add node globals to ignore undefined
+            globals: {
+                "__dirname": false,
+                "console": false,
+                "module": false,
+                "process": false,
+                "require": false,
+                "setTimeout": false,
+            }
+        },
         rules: {
             // Unused vars reported as error
             "@typescript-eslint/no-unused-vars": [
@@ -19,7 +30,6 @@ export default tseslint.config(
             // Required to import JS modules
             "@typescript-eslint/no-var-requires": "off",
             // TODO: Remove ignored rules and fix the code
-            "no-undef": "off",
             "no-unreachable": "off",
             "no-useless-catch": "off",
         }

examples/using-domains/app.js

@@ -46,6 +46,7 @@ appDomain.run(function () {
       // Or we can deal with it in our "Fake Error Handler" below
 
       // This will throw an error as fakeErrorHandler doesn't exist
+      // eslint-disable-next-line
       fakeErrorHandler.DealWith(err);
     }
   });