`mage test:all` should produce a combined code coverage result

authzed / spicedb

Open Source, Google Zanzibar-inspired database for scalably storing and querying fine-grained authorization data

https://authzed.com/docs

Apache License 2.0

5.12k stars 279 forks source link

`mage test:all` should produce a combined code coverage result #1348

Open jzelinskie opened 1 year ago

jzelinskie commented 1 year ago

If we add the -coverprofile flag to ever go test func in magefiles/test.go, we can aggregate them all into a single coverage file in All().

This'll be useful to track (perfect coverage is a non-goal of the project), but is also required for submitting to awesome-go

sephialaureenciaa commented 3 months ago

Hi, i'm interested in contributing to this!

tstirrat15 commented 3 months ago

@sephialaureenciaa thanks! have you checked out the CONTRIBUTING file? That should be all you need to get started and we'd welcome a pull request. If there's anything about the contribution guidance that seems like it's missing or confusing we'd also like to hear about it.