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

Add code coverage support with coveralls #151

Merged
merged 1 commit into from Jul 8, 2014

Conversation

3 participants
@caniszczyk
Contributor

caniszczyk commented Jul 8, 2014

See: https://coveralls.io/r/twitter/finatra

Signed-off-by: Chris Aniszczyk zx@twitter.com

Add code coverage support with coveralls
See: https://coveralls.io/r/twitter/finatra

Signed-off-by: Chris Aniszczyk <zx@twitter.com>
@caniszczyk

This comment has been minimized.

Contributor

caniszczyk commented Jul 8, 2014

Lets see if this actually works.

@coveralls

This comment has been minimized.

coveralls commented Jul 8, 2014

Coverage Status

Changes Unknown when pulling 44c0915 on add-coveralls into * on master*.

@caniszczyk

This comment has been minimized.

Contributor

caniszczyk commented Jul 8, 2014

@capotej let me know if this is something you wouldn't mind having

@capotej

This comment has been minimized.

Contributor

capotej commented Jul 8, 2014

Neat! I'll merge this and add it to the website also.

On Tuesday, July 8, 2014, Chris Aniszczyk notifications@github.com wrote:

@capotej https://github.com/capotej let me know if this is something
you wouldn't mind having


Reply to this email directly or view it on GitHub
#151 (comment).

Julio Capote

capotej added a commit that referenced this pull request Jul 8, 2014

Merge pull request #151 from twitter/add-coveralls
Add code coverage support with coveralls

@capotej capotej merged commit 6052e81 into master Jul 8, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@caniszczyk

This comment has been minimized.

Contributor

caniszczyk commented Jul 8, 2014

💚

@caniszczyk caniszczyk deleted the add-coveralls branch Jul 8, 2014

cacoco pushed a commit that referenced this pull request May 13, 2015

Merge pull request #151 from twitter/add-coveralls
Add code coverage support with coveralls

cacoco pushed a commit that referenced this pull request May 14, 2015

Merge pull request #151 from twitter/add-coveralls
Add code coverage support with coveralls
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment