Skip to content

Add unicode pass/fail icons to markdown rule summary#69

Merged
jesse-black merged 2 commits intomainfrom
codex/add-unicode-checkmark-and-cross-to-summary
Apr 5, 2026
Merged

Add unicode pass/fail icons to markdown rule summary#69
jesse-black merged 2 commits intomainfrom
codex/add-unicode-checkmark-and-cross-to-summary

Conversation

@jesse-black
Copy link
Copy Markdown
Owner

Motivation

  • Improve the readability of the generated Markdown summary by prefixing rule statuses with clear unicode indicators ( / ) so CI/PR viewers can scan pass/fail rows more quickly.

Description

  • Render rule statuses in the Diff Coverage table as ✅PASS or ❌FAIL instead of plain PASS/FAIL by updating src/render/markdown.rs.
  • Update existing unit assertions in src/render/markdown.rs to expect the ❌FAIL text for failing rules and add a new unit test that verifies both ✅PASS and ❌FAIL rows are produced.
  • Update the CLI markdown integration expectation in tests/cli_interface.rs to match the new ✅PASS output format.

Testing

  • Ran cargo test render::markdown::tests --lib and all renderer tests passed (including the new renders_rule_status_with_unicode_icons test).
  • Ran full validation with cargo xtask validate, which completed successfully after recording a base ref with cargo run --bin covgate -- record-base to satisfy the validation check-run; validate reported all checks passed.
  • Ran cargo fmt as part of validation and formatting checks passed.

Codex Task

@jesse-black jesse-black merged commit ad050f2 into main Apr 5, 2026
3 checks passed
@jesse-black jesse-black deleted the codex/add-unicode-checkmark-and-cross-to-summary branch April 5, 2026 19:06
@jesse-black jesse-black added the enhancement New feature or request label Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant