Skip to content

[8.19] Revert dual EUI versions#249567

Merged
delanni merged 2 commits intoelastic:8.19from
delanni:revert-dual-eui-versions
Jan 22, 2026
Merged

[8.19] Revert dual EUI versions#249567
delanni merged 2 commits intoelastic:8.19from
delanni:revert-dual-eui-versions

Conversation

@delanni
Copy link
Contributor

@delanni delanni commented Jan 19, 2026

Summary

Revert "[8.19] Add conditional switching between EUI releases (#219818) (#221920)"

This reverts commit d7d5e7b.

Having 2 dependencies point to the same version of the same dependency on npm seems to wreck yarn's logic, and causes issues on bootstrap. We no longer need this parallel variant of the dependency, so we can just remove it.

@delanni delanni marked this pull request as ready for review January 19, 2026 15:55
@delanni delanni added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Jan 19, 2026
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #10 / Event filter form Details and Conditions should change description

Metrics [docs]

✅ unchanged

History

@tkajtoch tkajtoch self-requested a review January 22, 2026 11:15
Copy link
Member

@tkajtoch tkajtoch left a comment

Choose a reason for hiding this comment

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

I tested the changes locally and can confirm everything is now correctly imported from @elastic/eui (that points to the amsterdam-specific release) as it should. LGTM

@delanni delanni merged commit 1126213 into elastic:8.19 Jan 22, 2026
13 checks passed
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 release_note:skip Skip the PR/issue when compiling release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants