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

Fix tests on older rubies #369

Merged
merged 1 commit into from Jan 13, 2017
Merged

Fix tests on older rubies #369

merged 1 commit into from Jan 13, 2017

Conversation

seejohnrun
Copy link
Collaborator

Don't try to bring in new activesupport on versions of Ruby that don't
support it

Don't try to bring in new activesupport on versions of Ruby that don't
support it
@hwhelchel
Copy link

@seejohnrun thanks for taking a crack at this. I agree with your use of Gem::Version. After I submitted my PR here, I fixed another 2.4 issue on another gem here using that api.

Here is how Rails handled supporting 2.4 in rails 4.2: rails/rails#27473

@seejohnrun seejohnrun merged commit b5a25ab into master Jan 13, 2017
@seejohnrun seejohnrun deleted the test_fix branch January 13, 2017 14:12
@seejohnrun
Copy link
Collaborator Author

my pleasure
I'm merging this for now, can you rebase your branch on top of this?

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