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.

Sign up for GitHub

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

Jump to bottom

[7.17] [Maps] Update [email protected] #188121

Merged
merged 1 commit into from
Jul 12, 2024
Merged

[7.17] [Maps] Update [email protected] #188121

merged 1 commit into from
Jul 12, 2024

Conversation

jsanz
Copy link
Member

@jsanz jsanz commented Jul 11, 2024 •
edited
Loading

Related with #188099

Updates @elastic/ems-client to 7.17.3 which is an update in the library dependencies without any new features.

@jsanz
Copy link
Member Author

jsanz commented Jul 11, 2024

/ci

@jsanz jsanz closed this Jul 11, 2024
@jsanz jsanz reopened this Jul 11, 2024
@jsanz jsanz added release_note:skip Skip the PR/issue when compiling release notes v7.17.23 Feature:Maps labels Jul 11, 2024
@jsanz jsanz requested review from nickpeihl and a team July 11, 2024 15:17
@jsanz
Copy link
Member Author

jsanz commented Jul 11, 2024

/ci

@jsanz jsanz changed the title [Maps] Update [email protected] [7.17] [Maps] Update [email protected] Jul 11, 2024
@jsanz jsanz marked this pull request as ready for review July 11, 2024 15:51
nickpeihl
nickpeihl approved these changes Jul 11, 2024
View reviewed changes
Copy link
Member

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

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

lgtm!

code review

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
maps 744 745 +1
mapsEms 85 89 +4
visTypeVega 181 185 +4
total +9

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
maps 2.7MB 2.7MB -912.0B
mapsEms 237.6KB 244.6KB +7.1KB
visTypeVega 1.7MB 1.7MB +7.1KB
total +13.3KB

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
kibana 868 872 +4

@jsanz jsanz merged commit 4d78f78 into elastic:7.17 Jul 12, 2024
110 checks passed
@jsanz jsanz deleted the update/ems-client/7.17.3 branch July 12, 2024 13:39
adelisle pushed a commit to Makila-AI/kibana that referenced this pull request Aug 5, 2024
@jsanz @adelisle
[7.17] [Maps] Update [email protected] (elastic#188121)
d890e0c 
Related with elastic#188099

Updates `@elastic/ems-client` to
[7.17.3](https://github.com/elastic/ems-client/releases/tag/v7.17.3)
which is an update in the library dependencies without any new features.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickpeihl nickpeihl nickpeihl approved these changes

@jbudz jbudz jbudz approved these changes

Assignees
No one assigned
Labels
Feature:Maps release_note:skip Skip the PR/issue when compiling release notes v7.17.23
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jsanz @elasticmachine @nickpeihl @jbudz