search

benc / simplecov-teamcity-summary

SimpleCov formatter that prints coverage summary information to display in TeamCity.
MIT License
10 stars 6 forks source link

Require simplecov #5

Closed steveklebanoff closed 9 years ago

steveklebanoff commented 9 years ago

Explicitly require simplecov.

If we don't this, we will get uninitialized constant errors if we install simplecov with the default instructions of setting require: false

See issue https://github.com/benc/simplecov-teamcity-summary/issues/4

agross commented 9 years ago

:+1:

steveklebanoff commented 9 years ago

@benc thoughts?

benc commented 9 years ago

Oops, missed that one. Great addition, thanks!

steveklebanoff commented 9 years ago

No problem, thanks for the merge