Skip to content

Ruby build matrix updates #165

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

Merged
merged 4 commits into from
Jul 20, 2016
Merged

Ruby build matrix updates #165

merged 4 commits into from
Jul 20, 2016

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented Jul 14, 2016

Fixes #164 and #160

@JonRowe
Copy link
Member Author

JonRowe commented Jul 14, 2016

/cc @samphippen @myronmarston

@@ -32,11 +33,14 @@ matrix:
env: JRUBY_OPTS='--server -Xcompile.invokedynamic=false'
- rvm: jruby
env: JRUBY_OPTS='--server -Xcompile.invokedynamic=false'
- rvm: jruby-9.0.3.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why 9.0.3.0 instead of 9.1.2.0?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be honest, because that was the latest version I found, oops!

@JonRowe
Copy link
Member Author

JonRowe commented Jul 17, 2016

Updated across the PRs, rspec/rspec-support#291 is the most ready to be merged (and prerequisite of the others), mocks and core builds need some more tweaking which I'll try to do today

@JonRowe
Copy link
Member Author

JonRowe commented Jul 17, 2016

Oh and rspec/rspec-expectations#930 is also good to go having had the least problems!

@JonRowe
Copy link
Member Author

JonRowe commented Jul 19, 2016

Only rspec/rspec-core#2293 outstanding, it's green with caveats

@JonRowe
Copy link
Member Author

JonRowe commented Jul 20, 2016

Merging as all PRs are merged.

@JonRowe JonRowe merged commit 38d2eeb into master Jul 20, 2016
@JonRowe JonRowe deleted the ruby_build_matrix_updates branch July 20, 2016 01:14
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.

2 participants