Skip to content
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

Only load coveralls when running inside Travis CI #79

Merged
merged 1 commit into from Jun 1, 2014

Conversation

plexus
Copy link
Member

@plexus plexus commented Dec 29, 2013

Coveralls is a no-op outside of Travis anyway, and it messes with setting custom Simplecov formatters.

@krainboltgreene
Copy link
Member

Alright, so I actually use coveralls locally to get a simple coverage print out. Do you have an alternative?

@plexus
Copy link
Member Author

plexus commented Dec 30, 2013

I see, I didn't realize it did anything locally. You can leave it in then.
I'm using a custom formatter when running inside emacs, I can probably
figure out a better way for it to play together with coveralls.
On 29 Dec 2013 21:16, "Kurtis Rainbolt-Greene" notifications@github.com
wrote:

Alright, so I actually use coveralls locally to get a simple coverage
print out. Do you have an alternative?


Reply to this email directly or view it on GitHubhttps://github.com//pull/79#issuecomment-31329154
.

@krainboltgreene
Copy link
Member

I'm entirely all for using coveralls only on travis and a abetter one locally. It just shows me per-file %, not terribly useful.

@plexus
Copy link
Member Author

plexus commented Dec 30, 2013

If you're on Emacs I could pass some cool stuff I've been working on :)

2013-12-29-231012_624x433_scrot

@plexus
Copy link
Member Author

plexus commented Dec 30, 2013

Actually before I used simplecov (which is what Coveralls uses under the hood). By default it generates a HTML report with per-line coverage. Apart from having to go to a browser it's an elegant solution.

krainboltgreene pushed a commit that referenced this pull request Jun 1, 2014
Only load coveralls when running inside Travis CI
@krainboltgreene krainboltgreene merged commit 556ca0d into hamstergem:master Jun 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants