Skip to content
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

Update development_dependency to latest version #2763

Merged
merged 2 commits into from
Jan 10, 2020

Conversation

ganmacs
Copy link
Member

@ganmacs ganmacs commented Jan 7, 2020

Signed-off-by: Yuta Iwama ganmacs@gmail.com

Which issue(s) this PR fixes:

none

What this PR does / why we need it:

Update development_dependency to latest version

Docs Changes:

none

Release Note:

none

Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
@ganmacs ganmacs force-pushed the update-dev-dependency branch 10 times, most recently from 63e228e to ff029bc Compare January 10, 2020 02:07
@@ -281,7 +281,8 @@ def initialize
@text = ""
@preamble_source = ""
@preamble = nil
URI.open(LICENSE_URL) do |io|
uri = URI.parse(LICENSE_URL)
Copy link
Member Author

@ganmacs ganmacs Jan 10, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: this change breaks tests in ruby 2.4 due to NoMethodError(`open' called for URI:Module' ). however I couldn't get the error message since stdout was captured ....

@ganmacs
Copy link
Member Author

ganmacs commented Jan 10, 2020

I'm merging this PR since tests for ruby 2.4 are failing now. if you have any questions, feel free to ask!
cc. @repeatedly

@ganmacs ganmacs merged commit afdd907 into fluent:master Jan 10, 2020
@ganmacs ganmacs deleted the update-dev-dependency branch January 10, 2020 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant