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

travis: Add coveralls integration #16

Merged
merged 1 commit into from Oct 27, 2015

Conversation

simonjbeaumont
Copy link
Contributor

For this to work you'll need to enable your repo on coveralls.io and then hit rebuild on the Travis job.

I added a make coverage target which runs quite happily:

$ make coverage
...[ snip ]...
Summary:
 - 'binding' points: 61/122 (50.00%)
 - 'sequence' points: 5/22 (22.73%)
 - 'for' points: none
 - 'if/then' points: 5/11 (45.45%)
 - 'try' points: 2/6 (33.33%)
 - 'while' points: none
 - 'match/function' points: 3/47 (6.38%)
 - 'class expression' points: none
 - 'class initializer' points: none
 - 'class method' points: none
 - 'class value' points: none
 - 'toplevel expression' points: none
 - 'lazy operator' points: 2/2 (100.00%)
 - total: 78/210 (37.14%)
/local/work/code/ocaml-irc-client

Signed-off-by: Si Beaumont simon.beaumont@citrix.com

Signed-off-by: Si Beaumont <simon.beaumont@citrix.com>
@johnelse
Copy link
Owner

Fatal error: exception Failure("'curl -sLf -F json_file=@- https://coveralls.io/api/v1/jobs' exited with 22.")

@simonjbeaumont
Copy link
Contributor Author

Right. You need to enable the repo on coveralls. :)

-- simonjbeaumont@gmail.com (via iPhone)

On 27 Oct 2015, at 00:34, John Else notifications@github.com wrote:

Fatal error: exception Failure("'curl -sLf -F json_file=@- https://coveralls.io/api/v1/jobs' exited with 22.")

Reply to this email directly or view it on GitHub.

@simonjbeaumont
Copy link
Contributor Author

According to this page you've only enabled ocaml-crc https://coveralls.io/github/johnelse

@johnelse
Copy link
Owner

I'm sure I turned that on last night 🐴 🐴 oh well, let's try again :)

@johnelse
Copy link
Owner

It worked! 🤘

johnelse added a commit that referenced this pull request Oct 27, 2015
travis: Add coveralls integration
@johnelse johnelse merged commit 3ba126c into johnelse:master Oct 27, 2015
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