-
Notifications
You must be signed in to change notification settings - Fork 104
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
Make gelf compactable with ruby-2.4.0 #55
Conversation
@tessie Thanks for your contribution! Please sign the CLA for this repository at https://cla-assistant.io/graylog-labs/gelf-rb?pullRequest=55 so that we can merge this PR. Also, please add Ruby 2.4.0 to the |
@joschi I have actually signed the CLA agreement. But it is still showing pending. Can you please help me |
Thanks @joschi Signed the CLA also. There was an issue with email id associated with commit.This is fixed |
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.
@tessie Thanks!
:) |
* 'master' of https://github.com/graylog-labs/gelf-rb: Prevent mutation of notifier arguments Add Ruby 2.4 to list of supported Ruby versions (graylog-labs#60) Make library compatible with Ruby 2.4.0 (graylog-labs#55) Update links in README.md Add additional non-Ruby logger levels for wider compatibility (graylog-labs#52) Opportunistically use Yajl for json encoding (graylog-labs#53) Update .travis.yml to use actual repository (graylog-labs#58)
In ruby 2.4.0
FixNum is deprciated
Also json (1.8.3) is not compatible with 2.4.0