Coverage report #4553
Replies: 3 comments
-
@rubenochiavone I'm all for this. I think it's a good idea. I'm not sure if we really need to create another CircleCi job for that. Can't we use an existing one? Coverage should be the same right? |
Beta Was this translation helpful? Give feedback.
-
I'd love to see a PR that does this as well. Go for it! |
Beta Was this translation helpful? Give feedback.
-
Thanks for the feedback. Before opening the PR let me ask you, is using codecov.io ok? If so, could you create a
Definitively. We can improve on that during the PR |
Beta Was this translation helpful? Give feedback.
-
Solidus has an extensive test spec, but there is no report about code coverage.
Currently it is possible to know each sub-project code coverage report by doing
However, this information in divided between
rspec
s executions and doesn't report overall coverage.I'd suggest
README.md
WDYT?
PS.: I did some work in advance but I'd rather discuss it before opening a PR - see https://github.com/rubenochiavone/solidus/tree/coverage
Beta Was this translation helpful? Give feedback.
All reactions