-
Notifications
You must be signed in to change notification settings - Fork 28
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
Support jruby #54
Support jruby #54
Conversation
.travis.yml
Outdated
@@ -2,17 +2,14 @@ language: ruby | |||
cache: bundler | |||
sudo: false | |||
rvm: | |||
- 2.2 | |||
- 2.3 | |||
- 2.4 |
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.
[SHOULD] Ruby 2.4 is maintained yet.
EOL is 2020-03-31
https://www.ruby-lang.org/en/downloads/branches/
So I think this shouldn't be erased
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.
I see.
OK, I will revert removing old ruby CI.
.travis.yml
Outdated
@@ -2,17 +2,14 @@ language: ruby | |||
cache: bundler | |||
sudo: false | |||
rvm: | |||
- 2.2 |
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.
[IMO] 2.2 and 2.3 are removed from build matrix.
This is soft deprecate? (build matrix is dropped, but users are allowed to install rspec-parameterized
in ruby 2.2 and 2.3 )
or this is hard deprecate ? (we stop support in ruby 2.2 and 2.3)
In any case, I think that it is better to create another PR because it is different from the subject of PR (jruby support)
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.
I mean soft deprecation. Too many test cases waste travis's resources.
636f6c1
to
ab55669
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.
@joker1007 LGTM!
:goodpoem: |
ref. #49, #50