Skip to content

Add React ^16.0 as a peerDependency, add EuiErrorBoundary#198

Merged
cjcenizal merged 6 commits intoelastic:masterfrom
cjcenizal:chore/react-16-required
Dec 11, 2017
Merged

Add React ^16.0 as a peerDependency, add EuiErrorBoundary#198
cjcenizal merged 6 commits intoelastic:masterfrom
cjcenizal:chore/react-16-required

Conversation

@cjcenizal
Copy link
Contributor

@cjcenizal cjcenizal commented Dec 8, 2017

Now if you forget to import a component, you'll see an error like this in the docs UI (but more helpful):

image

Other changes:

  • Export test module

@cjcenizal cjcenizal changed the title React ^16.0 is a peer dependency, add EuiErrorBoundary Add React ^16.0 as a peerDependency, add EuiErrorBoundary Dec 8, 2017
@cjcenizal cjcenizal requested a review from zinckiwi December 8, 2017 04:27
@cjcenizal
Copy link
Contributor Author

Blocked by elastic/kibana#15469

"yo": "2.0.0"
},
"peerDependencies": {
"react": "^16.0.0-rc || ^16.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

We should move react and react-router from dependencies to devDependencies too, because at present these modules get installed when I do a regular install of @elastic/eui.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

@pugnascotia pugnascotia left a comment

Choose a reason for hiding this comment

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

LGTM

@cjcenizal cjcenizal merged commit d8b8bb4 into elastic:master Dec 11, 2017
@cjcenizal cjcenizal deleted the chore/react-16-required branch December 11, 2017 23:19
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.

2 participants