Add support for and #56

merged 1 commit into from Sep 18, 2013


None yet

4 participants


Also cleanup pep8 for checking with flake8 in travis.

willkg commented Sep 13, 2013

I'm a pretty big anti-fan of coveralls. Do we really have to add that? Does it really mean something useful for funfactory?


We certainly don't have to. I do however like coverage reports and knowing how PRs affect coverage. What about it do you dislike?

@pmac pmac Add support for and
Also cleanup pep8 for checking with flake8
in travis.

I've turned off auto-commenting in Any objections to merging this in? I can still remove coveralls completely if people see no value in it at all.

jgmize commented Sep 16, 2013

+1 for coveralls with auto-commenting disabled

@pmclanahan pmclanahan merged commit afaa493 into mozilla:master Sep 18, 2013

1 check passed

Details default The Travis CI build passed
@pmclanahan pmclanahan deleted the pmclanahan:add-travis-coveralls branch Sep 18, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment