Skip to content

Use github.com/afunix/gocov instead of archived repo - #20

Open
b-dean wants to merge 1 commit into
AlekSi:masterfrom
b-dean:fix-for-go-1.25
Open

Use github.com/afunix/gocov instead of archived repo#20
b-dean wants to merge 1 commit into
AlekSi:masterfrom
b-dean:fix-for-go-1.25

Conversation

@b-dean

@b-dean b-dean commented Sep 10, 2025

Copy link
Copy Markdown

Fixes #19

@b-dean

b-dean commented Sep 10, 2025

Copy link
Copy Markdown
Author

For anyone needing a workaround until this is merged, you can do something like:

go run github.com/afunix/gocov/gocov@latest convert example.coverprofile | \
  go run github.com/b-dean/gocov-xml@v1.2.1-bdd.0 > example.xml

I plan on deleting my fork once this is merged though.

tomponline added a commit to canonical/lxd that referenced this pull request Nov 18, 2025
…ng to go1.25 compatible deps (#16981)

This pull request reverts the recent switch from `gocov/gocov-xml` to
`gocover-cobertura` for converting coverage to Cobertura XML format.
`gocover-cobertura` requires multiple manual workarounds and has
ultimately left us in a worse state with inconsistent coverage.
Additionally, this pull request updates the dependencies to versions
that support go1.25. For `gocov-xml`, we're using a fork that supports
go1.25 as a temporary workaround until
AlekSi/gocov-xml#20 is merged.

My aim is to get back to consistent code coverage reporting, analyze
results while considering the addition of snap tests to see where we're
at, and we can go from there.
@kalinon

kalinon commented Jan 7, 2026

Copy link
Copy Markdown

Any updates on this?

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.

github.com/axw/gocov not installable under golang 1.25.0

2 participants