-
Notifications
You must be signed in to change notification settings - Fork 46
loosen activeadmin restriction to allow 2.0 #50
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
Conversation
Hi @iarie, sorry if the changelog is confusing. The ruby 2.3 support drop has not yet been released. The list of changes in the 2.0 release can be found mainly under the 2.0.0.rc1 entry. The fact that the 2.0.0 entry says "No changes" means that there were no user visible changes between 2.0.0.rc2 and 2.0.0 final. |
I'll take a look at this PR tomorrow 🌞 |
active_admin_datetimepicker.gemspec
Outdated
spec.require_paths = ["lib"] | ||
|
||
spec.add_dependency "activeadmin", "~> 1.1" | ||
spec.add_dependency "activeadmin", ">= 1.1", "< 3.x" |
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.
If the intention is to not allow future ActiveAdmin 3.0.0 prereleases, then the correct way to do that is:
spec.add_dependency "activeadmin", ">= 1.1", "< 3.a"
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.
Thank you, my intention was exactly like that
@deivid-rodriguez hi, yes but there are no major changes in rc1 section as well, so I thought this gem should work fine on AA 2.0 |
Ah, ok. Since you said that the main breaking change was dropping support for ruby 2.3, I thought you had misunderstood the changelog 👍. It sounds like the only thing needed to get CI green would be to drop Rails 4.2 support in this plugin just like AA did. |
Firstly, @iarie thanx for contribution. I've found what was the problem with ActiveAdmin 2.0 and fixed in followed commit. Then I decided to remove 2.3 ruby support due to: Ruby 2.3 status: eol For major security issues, all releases in the current major series, and the last release in the additional major series will receive patches and new versions. This is currently 5.2.x, 5.1.x, 5.0.x and 4.2.x. |
I'll do version bump after merging this PR. Before that I want to check it manually. |
Checked it manually. Works fine. @iarie @deivid-rodriguez If you do not have any other comments on this PR, I'll merge it. |
Thank you, @workgena! |
https://github.com/activeadmin/activeadmin/blob/master/CHANGELOG.md
Activeadmin 2.0 do not сontain any major changes except ruby 2.3 support drop, this gem should be fine