-
Notifications
You must be signed in to change notification settings - Fork 30
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
Remove 4.2 support #155
Remove 4.2 support #155
Conversation
elrayle
commented
Sep 25, 2017
- update gemspec to require at least Rails 5.0
- update Travis to test Rails 5.0 and 5.1
5357cdb
to
5a4fae1
Compare
README.md
Outdated
@@ -634,7 +634,7 @@ Commit your features into a new branch and submit a pull request. | |||
|
|||
## Compatibility | |||
|
|||
Currently, it is compatible with Rails 4.0 and 4.1 under both Ruby 2.0 and 2.1. | |||
Currently, it is compatible with Rails 5.0 and 5.1 under both Ruby 2.4.1. |
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.
Can we add Ruby 2.3 to the test matrix and to this README?
Otherwise, remove the word "both"
qa.gemspec
Outdated
@@ -17,7 +17,7 @@ Gem::Specification.new do |s| | |||
s.files = Dir["{app,config,db,lib}/**/*", "LICENSE", "Rakefile", "README.md"] | |||
s.test_files = Dir["spec/**/*"] | |||
|
|||
s.add_dependency "rails", ">= 4.2.0", "< 6.0" | |||
s.add_dependency "rails", "~> 5.0", "< 6.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.
Remove "< 6.0"
as redundant?
337d4ec
to
f4a59e6
Compare
f4a59e6
to
e869ec8
Compare
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.
Looks great! Glad to have all this up to date.