Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(outputs): Linter issues #11595

Merged
merged 12 commits into from
Aug 3, 2022
Merged

fix(outputs): Linter issues #11595

merged 12 commits into from
Aug 3, 2022

Conversation

srebhan
Copy link
Member

@srebhan srebhan commented Aug 2, 2022

This PR fixes linter issues in plugins/outputs. After applying this only two issues in plugins/outputs/socket_writer and plugins/outputs/syslog regarding deprecation of err.Temportary() remain. However, fixing those is not straight forward and should be tackled in another PR.

@telegraf-tiger telegraf-tiger bot added the fix pr to fix corresponding bug label Aug 2, 2022
@srebhan srebhan added the plugin/output 1. Request for new output plugins 2. Issues/PRs that are related to out plugins label Aug 2, 2022
@srebhan srebhan requested a review from zak-pawel August 2, 2022 14:24
@telegraf-tiger
Copy link
Contributor

telegraf-tiger bot commented Aug 2, 2022

@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Aug 3, 2022
@sspaink sspaink merged commit ac26d78 into influxdata:master Aug 3, 2022
reimda pushed a commit that referenced this pull request Aug 15, 2022
reimda pushed a commit that referenced this pull request Aug 16, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
fix pr to fix corresponding bug linter plugin/output 1. Request for new output plugins 2. Issues/PRs that are related to out plugins ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants