chore(CI): simplify testing strategy

Author: MichaelDeBoey

.codecov.yml

@@ -3,29 +3,58 @@ on:
   push:
     branches: [main]
   pull_request:
-    branches: [main]
   schedule:
     - cron: 0 0 * * 0
 
 jobs:
-  test:
-    name: Test
+  lint:
+    name: ⬣ Lint
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        node: [12.x, 10.x, 8.x]
     steps:
-      - name: Checkout
-        uses: actions/checkout@v1
-      - name: Install Node.js ${{ matrix.node }}
-        uses: actions/setup-node@v1
+      - name: 🛑 Cancel Previous Runs
+        uses: styfle/[email protected]
+
+      - name: ⬇️ Checkout repo
+        uses: actions/checkout@v3
+
+      - name: ⎔ Setup Node
+        uses: actions/setup-node@v3
         with:
-          node-version: ${{ matrix.node }}
-      - name: Install Packages
+          node-version: 18
+
+      - name: 📥 Install dependencies
         run: npm install
-      - name: Test
-        run: npm test
-      - name: Send Coverage
-        run: npm run -s codecov
-        env:
-          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+
+      - name: ▶️ Run format:check script
+        run: npm run format:check
+
+      - name: ▶️ Run lint script
+        run: npm run lint
+
+    test:
+      name:
+        🧪 Test (Node@${{ matrix.node }})
+      strategy:
+        matrix:
+          node: [8.0.0, 8, 10, 12, 14, 16, 18]
+      runs-on: ubuntu-latest
+      steps:
+        - name: 🛑 Cancel Previous Runs
+          uses: styfle/[email protected]
+
+        - name: ⬇️ Checkout repo
+          uses: actions/checkout@v3
+
+        - name: ⎔ Setup Node v${{ matrix.node }}
+          uses: actions/setup-node@v3
+          with:
+            node-version: ${{ matrix.node }}
+
+        - name: 📥 Install dependencies
+          run: npm install
+
+        - name: ▶️ Run test script
+          run: npm run test
+
+        - name: ⬆️ Upload coverage report
+          uses: codecov/codecov-action@v3

.github/workflows/ci.yml

@@ -23,7 +23,6 @@
     "@types/jsdom": "^12.2.4",
     "@types/mocha": "^5.2.2",
     "@types/node": "^12.6.8",
-    "codecov": "^3.5.0",
     "dts-bundle": "^0.7.3",
     "eslint": "^6.1.0",
     "jsdom": "^15.1.1",
@@ -44,16 +43,14 @@
     "build:rollup": "rollup -c",
     "build:dts": "dts-bundle --name regexpp --main .temp/index.d.ts --out ../index.d.ts",
     "clean": "rimraf .temp index.*",
-    "codecov": "nyc report -r lcovonly && codecov -t ${CODECOV_TOKEN} --disable=gcov",
-    "lint": "eslint scripts src test --ext .ts",
-    "pretest": "run-s build lint",
+    "lint": "eslint . --ext .ts",
+    "pretest": "npm run build build",
     "test": "nyc _mocha \"test/*.ts\" --reporter dot --timeout 10000",
     "update:test": "ts-node scripts/update-fixtures.ts",
     "update:unicode": "run-s update:unicode:*",
     "update:unicode:ids": "ts-node scripts/update-unicode-ids.ts",
     "update:unicode:props": "ts-node scripts/update-unicode-properties.ts",
-    "preversion": "npm test",
-    "version": "npm run -s build",
+    "preversion": "npm test && npm run -s build",
     "postversion": "git push && git push --tags",
     "prewatch": "npm run -s clean",
     "watch": "_mocha \"test/*.ts\" --require ts-node/register --reporter dot --timeout 10000 --watch-extensions ts --watch --growl"