Skip to content

Conversation

@ericberens
Copy link
Contributor

First time reading through the docs and noticed a few discrepancies. Mostly minor updates to page titles and maintaining consistency across section headers. Aside from those, there were a couple oddities in the Type/Index Inference documentation.

@ericberens
Copy link
Contributor Author

I haven't, but briefly, gone through the NEST Core -> Filter DSL sub-sections although some of the page titles are off as well. I can update those as well if preferred.

@Mpdreamz
Copy link
Member

Mpdreamz commented Jul 1, 2014

Documentation PR's have a special place in my heart 👍 many many thanks for this @ericberens

Any additional help is appreciated.

Do you mind signing the Elasticsearch CLA so that we may pull your bits in ?

http://www.elasticsearch.org/contributor-agreement/

@ericberens
Copy link
Contributor Author

I'll comb through the other sections and see what I can do. :)

You should have a CLA for me. I submitted one before I sent the pull request but let me know if for whatever reason it didn't go through and I'll re-submit.

gmarz added a commit that referenced this pull request Jul 8, 2014
@gmarz gmarz merged commit 43ac721 into elastic:develop Jul 8, 2014
@gmarz
Copy link
Contributor

gmarz commented Jul 8, 2014

Thanks for this @ericberens! Keep 'em coming 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants