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.

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

Update react-ace because UNSAFE_* warnings #2526

Merged
merged 2 commits into from
Nov 13, 2019

Conversation

bevacqua
Copy link
Contributor

Nothing fundamental changes. I took the liberty to upgrade to 7.0.5 which is the last non-breaking release of react-ace, as react-ace@8 comes with some breaking changes that are more involved, but that we don't need to get into really

Copy link
Contributor

@chandlerprall chandlerprall left a comment

Choose a reason for hiding this comment

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

Changes LGTM; Pulled & tested EuiCodeEditor locally

Copy link
Contributor

@thompsongl thompsongl left a comment

Choose a reason for hiding this comment

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

Build and local docs run look good

@bevacqua bevacqua merged commit f2cd60e into elastic:master Nov 13, 2019
@bevacqua bevacqua deleted the update-react-ace-because-warnings branch November 13, 2019 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants