Skip to content

[8.18] Remove react-is dependency (#230117)#230456

Merged
legrego merged 4 commits intoelastic:8.18from
legrego:backport/8.18/pr-230117
Aug 5, 2025
Merged

[8.18] Remove react-is dependency (#230117)#230456
legrego merged 4 commits intoelastic:8.18from
legrego:backport/8.18/pr-230117

Conversation

@legrego
Copy link
Copy Markdown
Member

@legrego legrego commented Aug 4, 2025

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

## Summary

Removes the `react-is` dependency in favor of a simple first-party
check.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 8d27fcf)

# Conflicts:
#	package.json
#	x-pack/platform/plugins/shared/lens/public/datasources/form_based/form_based.test.ts
#	yarn.lock
@legrego legrego added the backport This PR is a backport of another PR label Aug 4, 2025
@legrego legrego requested a review from kibanamachine as a code owner August 4, 2025 16:44
@legrego legrego added the backport This PR is a backport of another PR label Aug 4, 2025
@legrego legrego enabled auto-merge (squash) August 4, 2025 16:44
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
core 411 409 -2
fleet 1221 1219 -2
kibanaReact 272 270 -2
kibanaUtils 161 159 -2
security 616 614 -2
securitySolution 7068 7066 -2
total -12

Async chunks

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

id before after diff
core 63.0KB 60.9KB -2.1KB
fleet 1.8MB 1.8MB -2.1KB
kibanaReact 194.0KB 191.9KB -2.1KB
kibanaUtils 63.0KB 60.9KB -2.1KB
security 532.2KB 530.0KB -2.1KB
securitySolution 9.1MB 9.1MB -2.1KB
total -12.8KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-npmDll 6.0MB 6.0MB -4.3KB

Copy link
Copy Markdown
Contributor

@kc13greiner kc13greiner left a comment

Choose a reason for hiding this comment

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

Backport LGTM!

@legrego legrego merged commit 6383bf9 into elastic:8.18 Aug 5, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants