Skip to content

Conversation

@javanna
Copy link
Member

@javanna javanna commented Sep 19, 2019

API spec now use an object for the documentation field. _common was not updated yet. This commit updates _common.json and its corresponding parser.

Closes #46744

@javanna javanna added the :Core/Infra/REST API REST infrastructure and utilities label Sep 19, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@javanna
Copy link
Member Author

javanna commented Sep 19, 2019

cc @elastic/es-clients

API spec now use an object for the documentation field. _common was not updated yet. This commit updates _common.json and its corresponding parser.

Closes elastic#46744
Copy link
Member

@delvedor delvedor left a comment

Choose a reason for hiding this comment

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

LGTM, this should also be backported to 7.x :)

@javanna javanna force-pushed the spec/update_common_parsing_format branch from 2d90971 to 7f5f308 Compare September 23, 2019 09:31
@javanna javanna merged commit 74e3af5 into elastic:master Sep 23, 2019
javanna added a commit that referenced this pull request Sep 23, 2019
API spec now use an object for the documentation field. _common was not updated yet. This commit updates _common.json and its corresponding parser.

Closes #46744

Co-Authored-By: Tomas Della Vedova <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The JSON spec parser should support the new format

4 participants