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

[APM] Start moving to renderApp #52586

Closed
wants to merge 1 commit into from
Closed

Conversation

smith
Copy link
Contributor

@smith smith commented Dec 9, 2019

Extract app rendering to a renderApp method.

Use start core and plugins in context to be ready for getStartServices API.

@smith smith force-pushed the nls/render-app branch 2 times, most recently from ff54c68 to c946200 Compare December 12, 2019 23:42
@smith smith added release_note:skip Skip the PR/issue when compiling release notes v7.6.0 labels Dec 12, 2019
@smith smith marked this pull request as ready for review December 12, 2019 23:43
@smith smith requested a review from a team as a code owner December 12, 2019 23:43
Extract app rendering to a renderApp method.

Use start core and plugins in context to be ready for getStartServices API.
@smith
Copy link
Contributor Author

smith commented Dec 18, 2019

@elasticmachine test this please

@sorenlouv
Copy link
Member

@elasticmachine test this please

fyi: There is also the shorter version retest

@smith
Copy link
Contributor Author

smith commented Dec 18, 2019

@elasticmachine test this please

@smith
Copy link
Contributor Author

smith commented Dec 18, 2019

retest

1 similar comment
@smith
Copy link
Contributor Author

smith commented Dec 19, 2019

retest

@smith
Copy link
Contributor Author

smith commented Dec 19, 2019

This is a legit failure, because the licensing plugin is only available in setup deps but not start deps. Will revisit and fix after the holidays.

@elasticmachine
Copy link
Contributor

💔 Build Failed

History

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

@LeeDr
Copy link
Contributor

LeeDr commented Jan 22, 2020

We're past FF for 7.6.0 so if this isn't a bug fix, please bump labels to v7.7.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants