-
Notifications
You must be signed in to change notification settings - Fork 759
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 to react 16 #792
Update to react 16 #792
Changes from 6 commits
dccc8de
373f45a
68c9ed2
c616ee5
c87d890
3cf34fb
cb66e31
7e450ab
32c9df3
e594694
896b4fc
6b0b878
57f7adc
61462c5
aa65c48
4b73b9f
8ab2c7b
1ba1110
18faeed
a8a9cd2
8883be9
691d2aa
0c1e60c
d7adca6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,10 @@ | ||
var addons = {}; | ||
addons.TransitionGroup = require("react-addons-transition-group"); | ||
addons.CSSTransitionGroup = require("react-addons-css-transition-group"); | ||
addons.LinkedStateMixin = require("react-addons-linked-state-mixin"); | ||
addons.createFragment = require("react-addons-create-fragment"); | ||
addons.update = require("react-addons-update"); | ||
addons.PureRenderMixin = require("react-addons-pure-render-mixin"); | ||
addons.TransitionGroup = require("react-transition-group/TransitionGroup"); | ||
addons.CSSTransitionGroup = require("react-transition-group/CSSTransitionGroup"); | ||
addons.update = require("immutability-helper"); | ||
|
||
if (process.env.NODE_ENV !== "production") { | ||
addons.TestUtils = require("react-addons-test-utils"); | ||
addons.Perf = require("react-addons-perf"); | ||
addons.TestUtils = require("react-dom/test-utils"); | ||
} | ||
|
||
module.exports = addons; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
Todo = React.createClass | ||
Todo = createReactClass | ||
render: -> | ||
`<li>{this.props.todo}</li>` | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
Todo = React.createClass | ||
Todo = createReactClass | ||
render: -> | ||
`<li>{this.props.todo}</li>` | ||
|
||
|
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.
Is it important to distinguish between these two varieties of preprocessed JS? I mean, what's the scenario where you want
require(...)
but notimport
?Anyone who's using webpacker will probably already use Babel (for JSX), right? So do you think one of these options would be sufficient by itself?
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 was tempted to scrap all the old style generators but was not sure on the effect it would have with sprockets + babel transpiler. I did what looked safe given I still don't fully know the system and saw the
option[:es6]
used nearby in the same file.Will continue on this path a little this afternoon, seems like the most pressing issue for now, then maybe the HMR ( 793 ) thing after once I get time.