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

Set up logging -> Configure logging #6186

Merged

Conversation

roncohen
Copy link
Contributor

image

@dedemorton
Copy link
Contributor

dedemorton commented Jan 25, 2018

Sure, looks fine. TBH, I want to do more cleanup on this TOC, but it seems like a low priority. I'm not sure any user will be confused, though maybe you're suggesting "configure" because you think it will be better for SEO? If it's the inconsistency you're concerned about, that's true of other topics, too. Although "set up" leaves you fee to cover more than just configuration.

@dedemorton
Copy link
Contributor

Since you are in the files already...I am OK with you fixing the problem everywhere.

@ruflin ruflin added the review label Jan 26, 2018
@dedemorton
Copy link
Contributor

@roncohen Nevermind, I will just merge and fix the consistency later. thanks!

@dedemorton dedemorton closed this Jan 26, 2018
@dedemorton dedemorton reopened this Jan 26, 2018
@dedemorton dedemorton merged commit 3cb07cb into elastic:master Jan 26, 2018
@roncohen
Copy link
Contributor Author

thanks @dedemorton! I agree there are more inconsistencies. I just saw this as i was working to move it into APM docs.

@dedemorton dedemorton added needs_backport PR is waiting to be backported to other branches. and removed backport labels Jan 27, 2018
dedemorton pushed a commit to dedemorton/beats that referenced this pull request Jan 27, 2018
@dedemorton dedemorton removed the needs_backport PR is waiting to be backported to other branches. label Jan 27, 2018
ruflin pushed a commit that referenced this pull request Jan 27, 2018
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
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.

3 participants