-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Fix React v15.5.0 new deprecation warnings #1171
Conversation
@u9lyfish, thanks for your PR! By analyzing the history of the files in this pull request, we identified @warmhug, @liqi07 and @pingan1927 to be potential reviewers. |
Codecov Report
@@ Coverage Diff @@
## master #1171 +/- ##
==========================================
- Coverage 69.73% 69.72% -0.02%
==========================================
Files 218 218
Lines 4111 4122 +11
Branches 1217 1218 +1
==========================================
+ Hits 2867 2874 +7
- Misses 1243 1247 +4
Partials 1 1
Continue to review full report at Codecov.
|
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.
seems no problem, but codecov
...
@paranoidjk
@bccsafe just ignore codecov sometimes. If you reviewd this PR is ok, them merge it. |
* Fix PropTypes deprecation warnings https://facebook.github.io/react/blog/2017/04/07/react-v15.5.0.html * Fix createClass deprecation warnings https://facebook.github.io/react/blog/2017/04/07/react-v15.5.0.html
React v15.5.0 extracted
React.PropTypes
andReact.createClass
into their own packages. Usage of these two modules will result in console errors.See: https://facebook.github.io/react/blog/2017/04/07/react-v15.5.0.html
This PR migrates components to new styles according to official guide above.