Skip to content

Conversation

@mshustov
Copy link
Contributor

@mshustov mshustov commented Aug 6, 2020

Summary

Babel v7.11 supports new syntax introduced in TSv4
Part of #71932

@mshustov mshustov added chore Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// Team:Operations Kibana-Operations Team release_note:skip Skip the PR/issue when compiling release notes labels Aug 6, 2020
@mshustov mshustov requested review from a team as code owners August 6, 2020 10:11
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-platform (Team:Platform)

@elasticmachine
Copy link
Contributor

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

@joshdover joshdover mentioned this pull request Aug 6, 2020
3 tasks
@mshustov mshustov mentioned this pull request Aug 6, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

async chunks size

id value diff baseline
console 1.1MB +12.0B 1.1MB
indexManagement 1.5MB +64.0B 1.5MB
ml 7.9MB -56.0B 7.9MB
securitySolution 7.3MB +94.0B 7.3MB
total +114.0B

page load bundle size

id value diff baseline
advancedSettings 980.7KB -6.0B 980.7KB
inputControlVis 295.7KB +64.0B 295.6KB
total +58.0B

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

@afharo
Copy link
Member

afharo commented Aug 6, 2020

I noticed there are changes in packages/kbn-i18n/package.json. Shouldn't @elastic/kibana-localization be pinged as a codeowner (it looks like a CODEOWNERS file issue unrelated to this PR)?

@mshustov mshustov requested a review from a team August 6, 2020 12:09
Copy link
Member

@afharo afharo left a comment

Choose a reason for hiding this comment

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

Telemetry changes LGTM!
File: packages/kbn-analytics/package.json

Tested and the UI analytics works great

Copy link
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

@mshustov mshustov merged commit 47fdd59 into elastic:master Aug 6, 2020
@mshustov mshustov deleted the bump-babel-deps branch August 6, 2020 20:00
mshustov added a commit to mshustov/kibana that referenced this pull request Aug 6, 2020
mshustov added a commit that referenced this pull request Aug 7, 2020
* bump babel deps to support TS v4 (#74495)

# Conflicts:
#	yarn.lock

* update yarn lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported chore release_note:skip Skip the PR/issue when compiling release notes Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// Team:Operations Kibana-Operations Team v7.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants