Remove gocovmerge
The following discussion from !183 (merged) should be addressed:
-
@nick.thomas started a discussion: License management complains about this dependency. It's bsd 2-clause: https://github.com/wadey/gocovmerge/blob/master/LICENSE so we could just add it to
doc/dependency_decisions.yml
Alternatively, we could do like gitlab-elasticsearch-indexer and remove it altogether: https://gitlab.com/gitlab-org/gitlab-elasticsearch-indexer/blob/master/Makefile#L30
For some context - we used to run coverage for $(allpackages) one at a time, and then merge them all together at the end. This is because we couldn't use
go test -short ./...
as it included thevendor/
directory.We could now just do that and remove the "merge these coverage reports together" step that this dependency enables.