Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added code coverage tool (fixes #195) #196

Merged
merged 1 commit into from
Nov 9, 2019

Conversation

stanislawK
Copy link
Contributor

  • make start

Show code-coverage in terminal's output:
make code-cov

Generate code-coverage report to html
make cov-report
Then open from project directory htmlcov/index.html to see report details

@OtisRed
Copy link
Contributor

OtisRed commented Nov 6, 2019

@jacekkalbarczyk please review and merge

Copy link
Member

@magul magul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From my perspective code-cov and cov-report (I would even abandon these two and make them part of make test).

We could also push these coverage reports to codecov.io or coveralls as we did in Volontulo (this would also allow us to put badge in a readme). See: https://codecov.io/github/CodeForPoznan/volontulo?branch=master

@magul magul merged commit d747517 into CodeForPoznan:master Nov 9, 2019
@magul
Copy link
Member

magul commented Nov 9, 2019

One thought more here - moving into Codecov.io will also allow us to store coverage reports from backend and frontend in the same place in the same format.

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.

3 participants