Skip to content

Demo: coverage report comment for the go-coverage-report README - #6

Merged
fgrosse merged 1 commit into
readme-screenshot-basefrom
readme-screenshot
Sep 24, 2026
Merged

fgrosse merged 1 commit into
readme-screenshot-basefrom
readme-screenshot

Conversation

@fgrosse

@fgrosse fgrosse commented Sep 24, 2026

Copy link
Copy Markdown
Owner

Demo pull request to show the new coverage report comment of fgrosse/go-coverage-report#110 for the README screenshot. Not meant to be merged.

It changes the coverage in two packages: the new Clone methods are untested (decrease in prioqueue) and new tests cover topk.Largest (increase in topk).

Demo change to show the coverage report comment of
fgrosse/go-coverage-report#110 in action.
@github-actions

Copy link
Copy Markdown

Merging this branch changes the coverage (1 decrease, 1 increase)

Impacted Packages Coverage Δ 🤖
github.com/fgrosse/prioqueue 92.59% (-7.41%) 👎
github.com/fgrosse/prioqueue/topk 100.00% (+54.17%) 🌟
Coverage details

Compared to base commit 067c0f9 (run #36055790830)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/fgrosse/prioqueue/max_heap.go 92.59% (-7.41%) 54 (+4) 50 4 (+4) 👎
github.com/fgrosse/prioqueue/min_heap.go 92.59% (-7.41%) 54 (+4) 50 4 (+4) 👎
github.com/fgrosse/prioqueue/topk/topk.go 100.00% (+54.17%) 24 24 (+13) 0 (-13) 🌟

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files:

  • github.com/fgrosse/prioqueue/topk/topk_test.go

fgrosse added a commit to fgrosse/go-coverage-report that referenced this pull request Sep 24, 2026
The screenshot shows the new layout of the details section including the
baseline commit and run. It was taken from fgrosse/prioqueue#6.
@fgrosse
fgrosse merged commit f0e3f7f into readme-screenshot-base Sep 24, 2026
2 checks passed
@fgrosse
fgrosse deleted the readme-screenshot branch September 24, 2026 20:50
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.

1 participant