Refactored code and unit tests to account for significantly updated d…

…ependencies (#6)
babbel · Jul 21, 2023 · c8b1fb8 · c8b1fb8
1 parent e5c0869
commit c8b1fb8
Show file tree

Hide file tree

Showing 15 changed files with 1,075 additions and 4,567 deletions.
diff --git a/.github/workflows/publishWorkflow.yml b/.github/workflows/publishWorkflow.yml
@@ -10,7 +10,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: "16.17.0"
+          node-version: "18.16.1"
           registry-url: https://npm.pkg.github.com
       - id: super-cache
         uses: mangs/super-cache-action@v3
@@ -26,4 +26,4 @@ jobs:
       #     NODE_AUTH_TOKEN: ${{ github.token }}
       - uses: mangs/simple-release-notes-action@v2
         with:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          github_token: ${{ secrets.GITHUB_TOKEN }}
diff --git a/.github/workflows/pullRequestWorkflow.yml b/.github/workflows/pullRequestWorkflow.yml
@@ -8,7 +8,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: "16.17.0"
+          node-version: "18.16.1"
           registry-url: https://npm.pkg.github.com
       - id: super-cache
         uses: mangs/super-cache-action@v3

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,17 @@
 # Changelog
 
+## 2.0.1
+
+- Refactored code and unit tests to account for significantly updated dependencies
+- Vitest option `--coverage` removed from `test:unit` NPM script and moved into new NPM script `test:unit:coverage` because package `@vitest/coverage-v8` has a bug in it that causes messages to get logged to the console that should not be
+- Added ESLint rule to require type imports to be in their own group
+- Replaced Vitest coverage reporter `c8` with `v8` as necessitated by the migration to Vitest version `0.32.0`
+- `validate:environment` NPM script now executes its subtasks in parallel
+- Update GitHub Action workflow files to use Node.js version `18.16.1`
+- Update expected internal NPM version to `9.5.1`
+- Remove dev dependency `@vscode/ripgrep`, instead rely on locally-installed version because NPM package installs fail too often
+- Update dependencies to latest
+
 ## 2.0.0
 
 - Convert code to TypeScript

diff --git a/README.md b/README.md
@@ -1,6 +1,6 @@
 # NPM module `@babbel/rollbar-client.js`
 
-Tiny, modern Rollbar JavaScript client whose code is mostly lazy-loaded if and when an error occurs.
+Tiny, modern Rollbar TypeScript client whose code is mostly lazy-loaded if and when an error occurs.
 
 ## Background and Overview
 

diff --git a/config/eslint/eslintBaseConfig.json b/config/eslint/eslintBaseConfig.json
@@ -37,7 +37,9 @@
       {
         "groups": [
           ["builtin", "external"],
-          ["internal", "unknown", "parent", "sibling", "index", "object", "type"]
+          ["internal", "unknown", "parent", "sibling", "index"],
+          ["type"],
+          ["object"]
         ],
         "newlines-between": "always"
       }

diff --git a/config/eslint/eslintTypescriptConfig.json b/config/eslint/eslintTypescriptConfig.json
@@ -7,8 +7,11 @@
   "extends": [
     "./eslintBaseConfig.json",
     "eslint-config-airbnb-typescript/base",
-    "plugin:@typescript-eslint/recommended",
-    "plugin:@typescript-eslint/recommended-requiring-type-checking",
+    "plugin:@typescript-eslint/recommended-type-checked",
+    "plugin:@typescript-eslint/stylistic-type-checked",
     "eslint-config-prettier"
-  ]
+  ],
+  "rules": {
+    "@typescript-eslint/consistent-type-imports": "error"
+  }
 }
diff --git a/config/vite/vite.config.mts b/config/vite/vite.config.mts
@@ -21,7 +21,7 @@ export default defineConfig({
   test: {
     clearMocks: true,
     coverage: {
-      provider: 'c8',
+      provider: 'v8',
       reportsDirectory: './vitestCodeCoverage',
     },
     environment: 'happy-dom',