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

Fix travis #37

Merged
merged 1 commit into from
Nov 2, 2014
Merged

Fix travis #37

merged 1 commit into from
Nov 2, 2014

Conversation

klobucar
Copy link
Contributor

@klobucar klobucar commented Nov 1, 2014

@derekcollison This fixes coveralls. I tested the failing build again to make sure it wasn't a fluke and I changed some of the args in the goveralls command. I'm not sure why it started to fail on it's own, but I've made the change to have it no longer be broken and fail the test.

@derekcollison
Copy link
Member

LGTM

On Nov 1, 2014, at 1:26 PM, Jonathon Klobucar notifications@github.com
wrote:

@derekcollison https://github.com/derekcollison This fixes coveralls. I
tested the failing build again to make sure it wasn't a fluke and I changed
some of the args in the goveralls command. I'm not sure why it started to
fail on it's own, but I've made the change to have it no longer be broken

and fail the test.

You can merge this Pull Request by running

git pull https://github.com/apcera/nats fix-goveralls

Or view, comment on, or merge it at:

#37
Commit Summary

  • Fix travis

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#37.

klobucar added a commit that referenced this pull request Nov 2, 2014
@klobucar klobucar merged commit 31a83dc into master Nov 2, 2014
@klobucar klobucar deleted the fix-goveralls branch November 2, 2014 00:04
olegshaldybin pushed a commit to olegshaldybin/nats that referenced this pull request Aug 22, 2015
Should have been included from the coveralls instructions to know more.
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