Skip to content

Conversation

@mattbrictson
Copy link
Contributor

Bug #554 was specific to Ruby 2.5.0, which is a good reminder that 2.5.0 should be included in the Travis matrix.

@mattbrictson
Copy link
Contributor Author

I based this PR on 838a571 (i.e. prior to the #554 fix) just to prove to myself that Travis would have caught the bug. I'll rebase this on master once Travis completes.

@mattbrictson
Copy link
Contributor Author

... and it succeed in Travis anyway. I guess it must be a macOS-specific issue.

In any case, it is still probably worthwhile to add 2.5.0 to the build matrix.

Rebased.

@sds sds merged commit ea3907a into sds:master Feb 13, 2018
@sds
Copy link
Owner

sds commented Feb 13, 2018

Thanks for the PR, @mattbrictson. Very strange that you couldn't reproduce on Travis, but having 2.5 in the build matrix is a good thing regardless, so thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants