fix(ci): add Node 18 skip condition for c8 v11 coverage step

c8 v11 requires Node 20+ (engines: ^20.0.0 || >=22.0.0).
Node 18 now runs plain npm test on PRs to avoid engine mismatch.
Also removes redundant --exclude flag from test:coverage script.

Also: fix ROADMAP.md progress table alignment (rows 7-12), mark
Phase 7 complete, add requirements-completed to 09-01-SUMMARY.md.
This commit is contained in:
Ethan Hurst
2026-02-25 21:13:35 +10:00
parent 898b82dee0
commit 896499120b
3 changed files with 44 additions and 2 deletions

View File

@@ -46,6 +46,6 @@
"build:hooks": "node scripts/build-hooks.js",
"prepublishOnly": "npm run build:hooks",
"test": "node --test tests/*.test.cjs",
"test:coverage": "c8 --check-coverage --lines 70 --reporter text --include 'get-shit-done/bin/lib/*.cjs' --exclude 'get-shit-done/bin/gsd-tools.cjs' --exclude 'tests/**' --all node --test tests/*.test.cjs"
"test:coverage": "c8 --check-coverage --lines 70 --reporter text --include 'get-shit-done/bin/lib/*.cjs' --exclude 'tests/**' --all node --test tests/*.test.cjs"
}
}