-
Notifications
You must be signed in to change notification settings - Fork 30
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
Output #1
Conversation
0123be2
to
37fa3ad
Compare
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.
LGTM
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.
All looks fine, just one place in the explanations that I wouldn't have understood if I didn't have a previous idea on what it's about 😅 Perhaps, it could be rephrased a bit as it's one of the 1st things that will be read by anyone opening the repo?
@yorugac updated, added a title and a first period giving more context. WDYT? |
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.
Yes, it sounds much clearer now. LGTM 👍
The InfluxDB v2 output implementation
TODO
Not use the admin token in composeAdded a note comment about itUpdate the header in code files