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

Make mentions of lowercasing refer to a general section of the docs. #332

Merged
merged 4 commits into from
Feb 22, 2019

Conversation

webmat
Copy link
Contributor

@webmat webmat commented Feb 18, 2019

The goal here is that the descriptions make sense no matter where they're seen (e.g. Kibana tooltip, as much as in the actual docs).

Copy link
Contributor

@MikePaquette MikePaquette left a comment

Choose a reason for hiding this comment

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

LGTM

@webmat webmat force-pushed the rm-readme-references branch 2 times, most recently from 6d7d8c7 to 71c7efb Compare February 22, 2019 20:49
Mathieu Martin and others added 4 commits February 22, 2019 16:01
The goal here is that the sentence makes sense no matter where it's seen (e.g. Kibana tooltip, as much as in the actual docs).
@webmat webmat force-pushed the rm-readme-references branch from 71c7efb to 6a3e439 Compare February 22, 2019 21:05
@webmat webmat merged commit 825ee39 into elastic:master Feb 22, 2019
@webmat webmat deleted the rm-readme-references branch February 22, 2019 21:15
webmat added a commit to webmat/ecs that referenced this pull request Mar 5, 2019
…lastic#332)

The goal here is that the sentence makes sense no matter where it's seen (e.g. Kibana tooltip, as much as in the actual docs).
webmat added a commit that referenced this pull request Mar 5, 2019
… section of the docs. (#332) (#364)

The goal here is that the sentence makes sense no matter where it's seen (e.g. Kibana tooltip, as much as in the actual docs).
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