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

chore: run make docs with various GOOS #12503

Merged
merged 1 commit into from
Jan 18, 2023

Conversation

powersj
Copy link
Contributor

@powersj powersj commented Jan 13, 2023

In order to correctly update all READMEs during go generate, pass the primary three GOOS that we support. This way all README will get any necessary changes.

fixes: #12499

In order to correctly update all READMEs during go generate, pass the
primary three GOOS that we support. This way all README will get any
necessary changes.

fixes: influxdata#12499
@Hipska
Copy link
Contributor

Hipska commented Jan 13, 2023

According to the makefile, we also support GOOS=freebsd. I know we don't have BSD specific plugins yet, but who knows for the future.

@Hipska Hipska added the docs Issues related to Telegraf documentation and configuration descriptions label Jan 13, 2023
@powersj
Copy link
Contributor Author

powersj commented Jan 13, 2023

According to the makefile, we also support GOOS=freebsd.

Right, I may need to go dig into this deeper and see which plugins are actually omitted with each GOOS. I know we have linux and windows only plugins, so we may only need to run those two.

@telegraf-tiger
Copy link
Contributor

@powersj powersj 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 Jan 17, 2023
Copy link
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks @powersj!

@srebhan srebhan merged commit 7725896 into influxdata:master Jan 18, 2023
srebhan pushed a commit that referenced this pull request Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore docs Issues related to Telegraf documentation and configuration descriptions 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.

make docs to build all docs
3 participants