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

Backport to 6.x: Multiple doc changes (#7696 #7883 #7888 #7889 #7903 #7937 #7939 #7948 #7949 #7950) #8188

Merged
merged 10 commits into from
Sep 4, 2018

Conversation

dedemorton
Copy link
Contributor

Replaces #8011

Cherry-picks the following doc PRs into the 6.x branch. Note that running make update resulted in some updates I wasn't expecting. Need someone to confirm that they are good for 6.x. Don't seem related the changes in cherry-picked PRs.

#7696
#7883
#7888
#7889
#7903
#7937
#7939
#7948
#7949
#7950

dedemorton and others added 10 commits August 31, 2018 11:23
* Add docs about append_fields
* Remove experimental tag from setup.template.json.enabled description
* Metricbeat: Add compatibility notes where missing, based on integration tests

* Added review suggestions
{ needs escaping as otherwise asciidoc will show an empty code block.
* Add document for beat export dashboard

Follow up from elastic#7239

* move to command reference

* address review comments

* review comments applied

* apply review feedback
I've been told that using "golang" instead of "Go" is considered a faux pas in some circles. :-)

I've replaced instances in the external docs, but I did not update the changelog, readmes, or code files.
@dedemorton
Copy link
Contributor Author

@jsoriano For this PR, running make update doesn't result in any updates...so I guess I must have done something wrong with the other PR. Hopefully that means this one will pass CI. :-)

@jsoriano
Copy link
Member

jsoriano commented Sep 3, 2018

Ok, failures in test don't seem related now 😃 LGTM

@dedemorton dedemorton merged commit abdb6d0 into elastic:6.x Sep 4, 2018
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