-
Notifications
You must be signed in to change notification settings - Fork 69
add SimpleCov #459
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
add SimpleCov #459
Conversation
Current build failed on Travis, but it seems totally unrelated to this PR. All feature ( |
@Morantron KEASECHO |
@Morantron @enricostano I started a branch to try fix this 👉 #461 First of all, I just upgraded gems to latest versions (basically because the |
@sauloperez friendly ping 😸 |
Not bad |
@markets @sauloperez are we going to add coverage check in CI? |
We should. Otherwise, coverage is not that useful. |
I created an issue for that #476 |
💎 https://github.com/colszowka/simplecov
Very useful 👌 as stated by @sauloperez in #281. I think we can configure Travis with Code Climate later.
After running the tests, you can open the generated
PROJECT_ROOT/coverage/index.html
file in a browser and you can see a detailed coverage report ✨At this point 62.58% covered.