search

kafkaex / kafka_ex

Kafka client library for Elixir
MIT License
596 stars 162 forks source link

Remove Coveralls #423

Closed dantswain closed 3 years ago

dantswain commented 3 years ago

We're not really paying attention to it and it causes spurrious test failures

I left the coveralls dependency because it generates a nice coverage report locally and personally I do find that useful.

jbruggem commented 3 years ago

@dantswain do you want to merge this ?

dantswain commented 3 years ago

Ah thanks for reminding me. I think I had sort of shelved this because I was investigating alternatives to travis-ci, but that stalled out, so might as well do this to help get rid of some spurious build failures.