Skip to content

Add a "coverage" target to the Makefile that generates a coverage report.#375

Closed
albu-diku wants to merge 1 commit intonextfrom
test/coverage-reporting
Closed

Add a "coverage" target to the Makefile that generates a coverage report.#375
albu-diku wants to merge 1 commit intonextfrom
test/coverage-reporting

Conversation

@albu-diku
Copy link
Copy Markdown
Contributor

No description provided.

@jonasbardino jonasbardino added the test-only Improvements or additions solely for better test coverage - without functionality changes label Oct 29, 2025
@jonasbardino
Copy link
Copy Markdown
Contributor

Looks good so far. You marked it draft so I'm not sure if you intend to expand it, but I think we can merge already if it runs.

@albu-diku
Copy link
Copy Markdown
Contributor Author

@jonasbardino it does :)

I marked it draft mostly as a hedge against the naming of the targets - I’m not sure I’m completely happy.

Perhaps for a first pass it’s better just to have only coverage and it generate the report - if split up happens later it would be elaboration for a specific reason.

What do you think?

@jonasbardino
Copy link
Copy Markdown
Contributor

@jonasbardino it does :)

I marked it draft mostly as a hedge against the naming of the targets - I’m not sure I’m completely happy.

Perhaps for a first pass it’s better just to have only coverage and it generate the report - if split up happens later it would be elaboration for a specific reason.

What do you think?

I'm not so worried about additional make targets but perhaps the most used should indeed just be called coverage. I haven't tried the html report format and would perhaps expect that to be a special case compared to default plain text; could be a different target or variable with default.

@albu-diku albu-diku closed this Apr 23, 2026
@jonasbardino
Copy link
Copy Markdown
Contributor

Did you mean to close this one? I think it's useful and wanted to merge in some form or another but awaited further input or status from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-only Improvements or additions solely for better test coverage - without functionality changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants