Skip to content

fix(NA): wrongly added include key on tsconfig.json for @kbn/monaco#118210

Merged
mistic merged 2 commits intoelastic:mainfrom
mistic:fix-windows-build-upon-kbn-monaco-changes
Nov 11, 2021
Merged

fix(NA): wrongly added include key on tsconfig.json for @kbn/monaco#118210
mistic merged 2 commits intoelastic:mainfrom
mistic:fix-windows-build-upon-kbn-monaco-changes

Conversation

@mistic
Copy link
Copy Markdown
Contributor

@mistic mistic commented Nov 10, 2021

This PR changes the layout of @kbn/monaco so we can have the build working on Windows that is broken since #109233

@mistic mistic added chore release_note:skip Skip the PR/issue when compiling release notes v8.1.0 Team:Operations Kibana-Operations Team labels Nov 10, 2021
@mistic mistic requested a review from jbudz November 10, 2021 19:29
@mistic mistic marked this pull request as ready for review November 10, 2021 19:30
@mistic mistic requested a review from a team as a code owner November 10, 2021 19:30
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@mistic mistic requested a review from sebelga November 10, 2021 19:30
Copy link
Copy Markdown
Contributor

@cjcenizal cjcenizal left a comment

Choose a reason for hiding this comment

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

Code LGTM. Didn't test locally.

@mistic mistic added the backport:skip This PR does not require backporting label Nov 10, 2021
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@mistic mistic merged commit 264bb7a into elastic:main Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting chore release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v8.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants