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

Update Ruby version constraints to allow Ruby v2.6 #12

Merged
merged 3 commits into from
Feb 1, 2019

Conversation

FinnWoelm
Copy link
Contributor

@FinnWoelm FinnWoelm commented Jan 17, 2019

Hi there,

Great gem! I just updated to Ruby v2.6, but acts_as_hashids is unhappy with my update. I pulled this repo and ran all tests locally with Ruby v2.6 and everything appears fine.

Thoughts? 😁

Update gemspec to allow Ruby versions >= 2.2.2 and < 2.7.
Also update Travis to test for Ruby v2.6
"On January 3rd 2019 the Bundler team released Bundler 2.0 which
dropped support for Ruby versions 2.2 and older"
(Travis, https://docs.travis-ci.com/user/languages/ruby/#bundler-20).
To make tests pass for Ruby v2.2, explicitly require installation of
Bundler v1.x.
When running tests for Ruby v2.6, allow tests with ActiveRecord 4.0 and
ActiveRecord 4.1 to fail. This mirrors the setup for Ruby v2.4 and
Ruby v2.6.
@FinnWoelm
Copy link
Contributor Author

Not sure why code coverage decreased 😭

@FinnWoelm
Copy link
Contributor Author

Hey @dtaniwaki — just following up on this. Any chance of merge? 🙂

@FinnWoelm
Copy link
Contributor Author

@dtaniwaki — any update on this? Would love to keep using this official repository rather than creating/maintaining my own fork of it.

Copy link
Owner

@dtaniwaki dtaniwaki left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you for your contribution.

@dtaniwaki dtaniwaki merged commit ab6a57d into dtaniwaki:master Feb 1, 2019
@FinnWoelm FinnWoelm deleted the allow-ruby-v2.6 branch February 2, 2019 01:15
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