Add support for test coverage merging; e.g. gocov test ./... about gocov HOT 8 CLOSED

I wrote a rough implementation; it implements support for:

gocov test ./... | gocov report

and other use like "./foo/...". Usage like:

gocov test ./isolate | gocov report

is unaffected by this change. The implementation is rough because it only look at the first argument.

from gocov.

I'd be fine with you proposing such a change to gocov.

Re merging, there are functions in gocov/gocovutil for merging gocov's format. It may be simpler to convert each profile to gocov's format first, then merge them together.

from gocov.

Maybe you missed the fact that I did attach a commit to this issue? https://github.com/maruel/gocov/commit/1b7da7b8665a89fb6d26bd25ff9eaed27891bb32

from gocov.

@maruel Indeed, I did, sorry. I will review and get back to you - travelling, so may take me a few days to respond.

from gocov.

Np, I'll be travelling in the bay area next week so I may also be slow to respond too.

from gocov.

Any update?

from gocov.

@maruel In case you missed the notification, and you're still interested in getting this merged, please see my comments on #65.

from gocov.

Thanks, I had totally missed the comments, will address ~tomorrow.

from gocov.