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

Add rails 5.2 compatibility #17

Closed
wants to merge 1 commit into from
Closed

Add rails 5.2 compatibility #17

wants to merge 1 commit into from

Conversation

emaglio
Copy link

@emaglio emaglio commented Jun 14, 2018

Hi there,
a client of mine is using this gem and we need to update to rails 5.2 so I was trying to update all the dependencies and found that this gem is not compatible with rails 5.2 yet. Because we need to update to rails urgently I thought sending a PR would make things happening quicker.

I think I have updated everything in the correct way but locally I'm not able to run the test and double check it (not really familiar with your gems yet). So I was hoping to send this PR and let travis to double check everything and maybe get some feedbacks from you guys to fix what needs to be fixed

Thanks
Cheers

@urkle
Copy link
Member

urkle commented Dec 15, 2018

Closing in favor of #16

@urkle urkle closed this Dec 15, 2018
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