Skip to content

[DOCS] Re-enable code snippet testing in close anomaly detection job API #48259

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

Merged
merged 2 commits into from
Oct 28, 2019

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Oct 18, 2019

The current code snippet tests in the documentation do not seem to support the use of non-OSS APIs and result in "rest api xxx doesn't exist in the rest spec" failures. This PR therefore tries to use "raw" methods in the build.gradle to re-enable testing in a machine learning API, similar to what is done for rollup and transform APIs (#45154).

@lcawl lcawl added >docs General docs changes WIP :ml Machine learning v8.0.0 v7.5.0 v7.6.0 v7.4.2 labels Oct 18, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@lcawl lcawl removed the WIP label Oct 18, 2019
@lcawl lcawl marked this pull request as ready for review October 18, 2019 21:10
@lcawl lcawl requested a review from szabosteve October 18, 2019 21:10
Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

LGTM

@lcawl lcawl merged commit 29ac34a into elastic:master Oct 28, 2019
@lcawl lcawl deleted the snippets1 branch October 28, 2019 15:08
lcawl added a commit to lcawl/elasticsearch that referenced this pull request Oct 28, 2019
lcawl added a commit to lcawl/elasticsearch that referenced this pull request Oct 28, 2019
lcawl added a commit to lcawl/elasticsearch that referenced this pull request Oct 28, 2019
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.

4 participants