-
Notifications
You must be signed in to change notification settings - Fork 369
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
Add JRuby 9.2 to CI #995
Add JRuby 9.2 to CI #995
Conversation
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 good, one small consideration regarding dependencies but this is not a blocker.
Feel free to merge. Thanks @marcotc !
@@ -51,8 +51,11 @@ Gem::Specification.new do |spec| | |||
spec.add_development_dependency 'yard', '~> 0.9' | |||
spec.add_development_dependency 'webmock', '~> 2.0' | |||
spec.add_development_dependency 'builder' | |||
spec.add_development_dependency 'ruby-prof' | |||
spec.add_development_dependency 'sqlite3', '~> 1.3.6' | |||
if RUBY_PLATFORM != 'java' |
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.
Not something we have to address here, but we should consider moving these "development dependencies" entirely out of the gemspec to the Gemfile
, as they drive our test suite mostly. (A change that would be analogous to what we did for pry-nav
.)
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.
Sounds like a nice way to clean up the gemspec file 👍
@@ -55,6 +55,8 @@ def flush | |||
def gc_metrics | |||
Hash[ | |||
GC.stat.map do |k, v| | |||
next if v.is_a?(Hash) # TODO: JRuby supports additional nested metrics |
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.
Will be interesting to expand on this later.
@@ -0,0 +1,15 @@ | |||
module PlatformHelpers |
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 dig this.
This PR adds basic support for running the tracer tests against JRuby.
Currently, we only run tracer core tests: integrations are not tested.