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.

Sign up for GitHub

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

Jump to bottom

Suppress warning #2652

Merged
merged 1 commit into from
Oct 15, 2019
Merged

Suppress warning #2652

merged 1 commit into from
Oct 15, 2019

Conversation

ganmacs
Copy link
Member

@ganmacs ganmacs commented Oct 15, 2019 •
edited
Loading

Which issue(s) this PR fixes:
no

What this PR does / why we need it:

Suppress the following warnings.

https://travis-ci.org/fluent/fluentd/jobs/598006587

instance variable @id not initialized
instance variable @timeout_checker not initialized
instance variable @sock not initialized

Docs Changes:

no need

Release Note:

no need

cc @repeatedly

@ganmacs
suppress warning
4ce7f37 
Signed-off-by: Yuta Iwama <[email protected]>
@ganmacs ganmacs self-assigned this Oct 15, 2019
@repeatedly repeatedly merged commit 24a0fea into fluent:master Oct 15, 2019
@repeatedly
Copy link
Member

Looks good

@ganmacs ganmacs deleted the suppress-warning branch October 16, 2019 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ganmacs ganmacs

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ganmacs @repeatedly