fix: address prior PR feedback — CI mutation grep, docs, eval DAG#42
fix: address prior PR feedback — CI mutation grep, docs, eval DAG#42haasonsaas merged 1 commit intomainfrom
Conversation
…ment - CI (PR40): Extract missed count from cargo-mutants summary correctly. Grep was taking first number (total mutants) instead of number before ' missed'; use sed to parse 'N missed' so baseline check is accurate. - README: Add PR workflow note under Contributing. - docs/mutation-testing: Note state/storage_pg not in CI; add local-run table and re-run commands. - eval DAG (PR39): Comment that ExpectationMatching/CommentCount/Benchmark form a linear chain so parallelizable hint has no effect today; keep for consistency. Made-with: Cursor
PR SummaryLow Risk Overview Updates contributor docs to clarify the PR workflow and to document that mutation testing for Written by Cursor Bugbot for commit 39e5f9b. This will update automatically on new commits. Configure here. |
Summary
2from14 mutants ... 2 missed, 9 caught), not the first number (14). The previousgrep -oE '[0-9]+' | head -1picked total mutants and caused spurious baseline failures.*state*and*storage_pg*are not run in CI; add table from local run and re-run commands.parallelizable: truehas no effect today; keep for consistency.Test plan
cargo test(no code behavior change beyond comment).Made with Cursor