switch coverall to codecov

Author: MoOx

.github/workflows/test.yml

@@ -1,40 +1,27 @@
-name: test
-on:
-  push:
-    branches:
-      - "main"
-      - "master"
-  pull_request:
+name: Test
 
-defaults:
-  run:
-    shell: bash
+on: [push, pull_request]
 
 jobs:
   test:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
     strategy:
-      fail-fast: false
+      fail-fast: true
       matrix:
-        # Maintained Node.js LTS lines. The published es5 dist still runs on
-        # older Node, but the test runner (node:test) requires Node >= 18.
-        node: [22, 24]
+        os: [ubuntu-latest, windows-latest]
+        node-version: [lts/*, latest]
     steps:
       - uses: actions/checkout@v4
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v4
         with:
-          fetch-depth: 1
-      - uses: actions/setup-node@v4
+          node-version: ${{ matrix.node-version }}
+      - run: npm ci
+      - run: npm test
+      - name: Upload coverage to Codecov
+        if: matrix.os == 'ubuntu-latest' && matrix.node-version == 'lts/*'
+        uses: codecov/codecov-action@v5
         with:
-          node-version: ${{ matrix.node }}
-
-      - name: npm ci
-        run: npm ci --ignore-scripts
-
-      - name: test
-        run: npm run test:coverage
-
-      - name: lint
-        run: npm run lint
-
-      - name: typecheck
-        run: npm run typecheck
+          token: ${{ secrets.CODECOV_TOKEN }}
+          files: ./coverage/lcov.info
+          fail_ci_if_error: false

.gitignore

@@ -1,6 +1,5 @@
-.nyc_output
 coverage
 node_modules
 dist
 .idea
-.vscode
+.vscode

package.json

@@ -28,10 +28,12 @@
     "lint:fix": "oxlint --fix",
     "format": "oxfmt src",
     "format:check": "oxfmt --check src",
+    "pretests": "npm run build",
+    "tests": "node --test src/__tests__/*.mjs",
+    "precoverage": "npm run build",
+    "coverage": "c8 node --test src/__tests__/*.mjs",
     "pretest": "npm run build",
-    "test": "node --test src/__tests__/*.mjs",
-    "pretest:coverage": "npm run build",
-    "test:coverage": "c8 node --test src/__tests__/*.mjs",
+    "test": "npm run lint && npm run typecheck && npm run tests && npm run coverage",
     "prepare": "npm run build",
     "release": "npx npmpub"
   },
@@ -66,7 +68,8 @@
     ],
     "reporter": [
       "text",
-      "text-summary"
+      "text-summary",
+      "lcovonly"
     ],
     "all": true
   }

src/__tests__/exceptions.mjs

@@ -22,6 +22,6 @@ throws('bad syntax', '-moz-osx-font-smoothing: grayscale');
 throws('bad syntax (2)', '! .body');
 
 throws('missing backslash for semicolon', '.;');
-throws('missing backslash for semicolon (2)', '.;');
-throws('unexpected / foo', '-Option/root', "Unexpected '/'. Escaping special characters with \\ may help.");
+throws('missing backslash for semicolon (2)', '.\;');
+throws('unexpected / foo', '-Option\/root', "Unexpected '/'. Escaping special characters with \\ may help.");
 throws('bang in selector', '.foo !optional', "Unexpected '!'. Escaping special characters with \\ may help.");