-
Notifications
You must be signed in to change notification settings - Fork 2
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
Bump dependencies for react 15.6 compatibility #2
Bump dependencies for react 15.6 compatibility #2
Conversation
You can also use the standalone build by including `react-select.js` and `react-select.css` in your page. (If you do this though you'll also need to include the dependencies.) For example: | ||
```html | ||
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react.min.js"></script> | ||
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/15.1.0/react-dom.min.js"></script> |
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.
Deleted this whole bit because I got annoyed at it matching a grep for older react versions. 🤐
@@ -1,7 +1,9 @@ | |||
import React from 'react'; | |||
import PropTypes from 'prop-types'; | |||
import createReactClass from 'create-react-class'; |
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.
A handful of these classes could be converted to es6 classes inheriting from React.PureComponent
but I didn't want to take the time to dig through and figure out which ones. May come back to that if I need to make more changes to do React 16 compatibility.
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.
lgtm
The diff is a bit noisy due to the dist files, but the important bits are:
package.json
to get deps compatible with 15.5/15.6.node-version
file to lock in a version where the build script works