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

Update travis list and add jruby #63

Merged
merged 1 commit into from May 30, 2016

Conversation

Projects
None yet
4 participants
@spk
Contributor

spk commented Mar 12, 2016

Update travis list
Cheers

@bensheldon

This comment has been minimized.

Contributor

bensheldon commented Mar 12, 2016

A larger build matrix will add some time to run the tests; seems like a change from 1min to 14min.

Also, the jruby build is both failing and ignored. It seems to be failing on installing rspec 3.0.

Edit: to add, I think we should add jruby to the test matrix and this is a good start. Thank you.

@spk spk force-pushed the spk:travis-updates branch from 483d22a to 46f3588 Mar 19, 2016

@coveralls

This comment has been minimized.

coveralls commented Mar 19, 2016

Coverage Status

Coverage decreased (-0.2%) to 99.091% when pulling 46f3588 on spk:travis-updates into 64e451c on RubyMoney:master.

@spk

This comment has been minimized.

Contributor

spk commented Mar 19, 2016

Hi @bensheldon, I've updated the PR with your suggestions.
I guess the coverage error is normal ?

@antstorm

This comment has been minimized.

Contributor

antstorm commented May 29, 2016

@spk can you please rebase it with master?

@spk spk force-pushed the spk:travis-updates branch from 46f3588 to 85a2379 May 29, 2016

@coveralls

This comment has been minimized.

coveralls commented May 29, 2016

Coverage Status

Coverage decreased (-0.2%) to 99.105% when pulling 85a2379 on spk:travis-updates into 7883071 on RubyMoney:master.

@spk

This comment has been minimized.

Contributor

spk commented May 29, 2016

@antstorm updated

@antstorm

This comment has been minimized.

Contributor

antstorm commented May 30, 2016

@spk great, thank you! Merging!

@antstorm antstorm merged commit d04ce7d into RubyMoney:master May 30, 2016

1 of 2 checks passed

coverage/coveralls Coverage decreased (-0.2%) to 99.105%
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment