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

Change titles in config reference topics #1885

Merged
merged 3 commits into from
Jun 21, 2016

Conversation

dedemorton
Copy link
Contributor

This PR resolves issue #1792

Section titles are renamed to match the section comments in the long version of each config file (hopefully making it easier for users to find what they're looking for in the reference docs).

Some content has been moved around to better match the organization of the config files.


You can specify configuration options that control Filebeat behavior at a global level.

==== Options
Copy link
Collaborator

Choose a reason for hiding this comment

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

I assume we need this title here, because asciidoc does not allow leaving it out?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ruflin It's not exactly an asciidoc issue. In most cases, we could remove this heading and move everything up a level, but then we'd get inconsistent formatting for options because in some places we break the options down into further categories. I think it's better to keep all the options at the same heading level so that they appear consistent across the doc (same formatting, etc), and so that you don't get a long list of options under the "On this page" area above the navigation.

@ruflin
Copy link
Collaborator

ruflin commented Jun 20, 2016

LGTM. I left some minor comments. We should defintively rename shipperconfig to prevent confusion.

@dedemorton
Copy link
Contributor Author

@ruflin I've updated the docs to reflect your feedback. This PR should be ready to merge if my changes look OK to you.

@ruflin ruflin merged commit 7576fcd into elastic:master Jun 21, 2016
@ruflin
Copy link
Collaborator

ruflin commented Jun 21, 2016

@dedemorton I merged it as is. About the run options: the file should probably be moved into the packetbeat docs folder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants