-
Notifications
You must be signed in to change notification settings - Fork 323
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
supporting activemodel >= 5.0 #392 #399
Conversation
Please merge this in. Gems that depend on Her and that are included in a Rails app are blocking upgrading to Rails 5. |
@@ -23,8 +23,8 @@ Gem::Specification.new do |s| | |||
s.add_development_dependency "fivemat", "~> 1.2" | |||
s.add_development_dependency "json", "~> 1.8" | |||
|
|||
s.add_runtime_dependency "activemodel", ">= 3.0.0", "<= 4.3.0" | |||
s.add_runtime_dependency "activesupport", ">= 3.0.0", "<= 4.3.0" | |||
s.add_runtime_dependency "activemodel", ">= 3.0.0", "<= 5.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be < 5.1.0
so patches to rails 5 are compatible?
If anyone really wants to use this, I've forked and created a new gem: https://rubygems.org/gems/her5 |
+1 |
1 similar comment
+1 |
hi @zkwentz @ryanswood @arturseletskiy @jkappers I've just got commit access to this repo and this will be my first merge! Will keep you posted here. |
Hi folks, Rather than accept this PR I've written a slightly more permissive change to the gemspec. I'll close this one now. Ed |
No description provided.