Closed zspencer closed 6 years ago
I think we don't need both Coveralls and CodeClimate for coverage reporting and should pick one. I'm leaning slightly towards Coveralls because its reports seem more granular, but don't have strong feelings.
Yep, I agree. Was hoping I could make it work with both but since it took more than 30m I'm going to remove that and rebase it down to one.
Resolves #20
This also turns on "rails" mode for Coveralls, which maybe we don't want to do? I was just getting nervous seeing that 100% covered despite knowing we've got uncovered stuff.
It appears codeclimate's coverage is incompatible with coveralls, but that's fine since coveralls notifies us when the number drops below a certain amount. It does mean we won't get to see the unified maintainability / coverage view in CodeClimate, but that's ok.