Skip to content

[Maps] Convert ES-sources to typescript#81951

Merged
thomasneirynck merged 77 commits intoelastic:masterfrom
thomasneirynck:maps/es_source_typescript3
Nov 5, 2020
Merged

[Maps] Convert ES-sources to typescript#81951
thomasneirynck merged 77 commits intoelastic:masterfrom
thomasneirynck:maps/es_source_typescript3

Conversation

@thomasneirynck
Copy link
Copy Markdown
Contributor

Summary

Chore TS-conversion.

Checklist

For maintainers

@thomasneirynck thomasneirynck requested a review from nreese November 2, 2020 22:34
@thomasneirynck thomasneirynck requested a review from nreese November 3, 2020 14:51
Copy link
Copy Markdown
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

Thanks for converting the sources to TS. This is a big chunk of logic

LGTM
code review

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
maps 619 620 +1

async chunks size

id before after diff
maps 2.8MB 2.8MB +3.8KB

page load bundle size

id before after diff
maps 153.9KB 153.9KB +24.0B

History

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

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

Labels

chore release_note:skip Skip the PR/issue when compiling release notes Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants