-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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.
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
[6.7] Upgrades EUI to 6.10.6 (backported browser-related fix) (#33883) #33972
Conversation
* eui to 6.10.6 * update snapshots related to onClick
💔 Build Failed |
💚 Build Succeeded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Didn't know if the react upgrades in the test files were ok for this version of kibana or if it was just a cleanup. Looks like a cleanup, but I see yarn.lock changed so want to make sure after that last backport.
"@elastic/eui": "5.0.0", | ||
"react": "^16.3.0" | ||
"@elastic/eui": "6.10.6", | ||
"react": "^16.6.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume these react upgrades are OK here @chandlerprall @thompsongl ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, those look fine 👍
Are we ok to merge to the |
Nah. We should wait for the release first. Should be later today or tomorrow. |
jenkins, test this |
💚 Build Succeeded |
Backports the following commits to 6.7: