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

Minor change to stack monitoring tooltips #31855

Merged
merged 5 commits into from
Mar 4, 2019

Conversation

cachedout
Copy link
Contributor

In the Linux documentation and source code, cgroups are always referred to without
capitalization and should be here as well.

Summary

Very minor change to grammar in the text of the Stack Monitoring application.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11

For maintainers

In the Linux documentation and source code, cgroups are always referred to without
capitalization and should be here as well.
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cachedout cachedout requested a review from a team March 2, 2019 00:18
@cachedout cachedout added the Team:Monitoring Stack Monitoring team label Mar 2, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/stack-monitoring

@cachedout
Copy link
Contributor Author

@chrisronline This is ready for review. No hurry though.

Copy link
Contributor

@chrisronline chrisronline left a comment

Choose a reason for hiding this comment

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

LGTM!

@cachedout cachedout merged commit 4e9b7ef into elastic:master Mar 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Monitoring Stack Monitoring team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants