Skip to content

src: refactor expression printing in code coverage report.#1370

Merged
nickg merged 1 commit intonickg:masterfrom
Blebowski:cover-highlight-multiline-expressions
Jan 17, 2026
Merged

src: refactor expression printing in code coverage report.#1370
nickg merged 1 commit intonickg:masterfrom
Blebowski:cover-highlight-multiline-expressions

Conversation

@Blebowski
Copy link
Copy Markdown
Contributor

This MR has two changes to the way coverage items are printed in the HTML coverage report:

  1. Line numbers are printed in the coverage item title, as opposed to only start of the code sample
  2. Expressions are printed within single line. Comments are filtered. This simplifies drawing of LHS/RHS indicators and allows to draw them also for multi-line expressions. This format matches how some of the commercial tools show expressions in coverage tools.

Very long expressions don't fit into single line, but I think that can be tolerated.

@Blebowski Blebowski force-pushed the cover-highlight-multiline-expressions branch from e69524f to 21d9746 Compare January 3, 2026 15:31
@Blebowski Blebowski force-pushed the cover-highlight-multiline-expressions branch from 21d9746 to b86a4ff Compare January 11, 2026 12:37
@Blebowski
Copy link
Copy Markdown
Contributor Author

This is now also ready for review. I ran the HTML validator over these changes too.

An example of "new format" in case of complex multi-line expressions:
image

vs. the old format:

image

@Blebowski Blebowski force-pushed the cover-highlight-multiline-expressions branch from b86a4ff to a79aa86 Compare January 13, 2026 20:47
@Blebowski
Copy link
Copy Markdown
Contributor Author

Hi @nickg, do you think these changes can be merged ?

@nickg nickg merged commit dad3032 into nickg:master Jan 17, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants