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.

Sign up for GitHub

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

Jump to bottom

Propose language changes in FAQ #10709

Merged
merged 1 commit into from
Mar 5, 2017
Merged

Propose language changes in FAQ #10709

merged 1 commit into from
Mar 5, 2017

Conversation

maxvanzoest
Copy link

I think I found some small language mistakes in the FAQ file, this pull request proposes some changes.

@elasticmachine
Copy link
Contributor

Can one of the admins verify this patch?

@maxvanzoest maxvanzoest mentioned this pull request Mar 5, 2017
Closed
@epixa epixa self-assigned this Mar 5, 2017
@epixa epixa added the review label Mar 5, 2017
@epixa
Copy link
Contributor

epixa commented Mar 5, 2017

jenkins, test this

@epixa epixa merged commit 06049c2 into elastic:master Mar 5, 2017
@epixa epixa changed the title Propose language changes Propose language changes in FAQ Mar 5, 2017
@elasticmachine
Copy link
Contributor

Can one of the admins verify this patch?

epixa pushed a commit that referenced this pull request Mar 5, 2017
@epixa
propose language changes (#10709)
707f774
@epixa
Copy link
Contributor

epixa commented Mar 5, 2017

Thanks for these tweaks!

@maxvanzoest maxvanzoest deleted the faq branch March 5, 2017 18:59
@maxvanzoest maxvanzoest restored the faq branch March 5, 2017 19:16
@maxvanzoest maxvanzoest deleted the faq branch March 5, 2017 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@epixa epixa

Labels
review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@maxvanzoest @elasticmachine @epixa