Skip to content

[ML] Functional tests - export service types#52612

Merged
pheyos merged 1 commit intoelastic:masterfrom
pheyos:export_types
Dec 11, 2019
Merged

[ML] Functional tests - export service types#52612
pheyos merged 1 commit intoelastic:masterfrom
pheyos:export_types

Conversation

@pheyos
Copy link
Copy Markdown
Member

@pheyos pheyos commented Dec 10, 2019

Summary

With this PR the types of the ML services that are used in other services are exported from the service file to allow a cleaner re-use.

This is following @spalger 's suggestion in #52167 (comment)

@pheyos pheyos requested a review from a team as a code owner December 10, 2019 10:52
@pheyos pheyos self-assigned this Dec 10, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

LGTM

@pheyos pheyos merged commit 3130759 into elastic:master Dec 11, 2019
@pheyos pheyos deleted the export_types branch December 11, 2019 15:25
pheyos added a commit to pheyos/kibana that referenced this pull request Dec 12, 2019
With this PR the types of the ML services that are used in other services are exported from the service file to allow a cleaner re-use.
pheyos added a commit to pheyos/kibana that referenced this pull request Dec 12, 2019
With this PR the types of the ML services that are used in other services are exported from the service file to allow a cleaner re-use.
pheyos added a commit that referenced this pull request Dec 12, 2019
With this PR the types of the ML services that are used in other services are exported from the service file to allow a cleaner re-use.
pheyos added a commit that referenced this pull request Dec 12, 2019
With this PR the types of the ML services that are used in other services are exported from the service file to allow a cleaner re-use.
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