Closed
Bug 1217979
Opened 9 years ago
Closed 9 years ago
Use react-dev as our only react lib
Categories
(DevTools :: General, defect)
Tracking
(firefox44 fixed)
RESOLVED
FIXED
Firefox 44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: jsantell, Assigned: jsantell)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jlong
:
review+
|
Details | Diff | Splinter Review |
Running into a lot of issues with react-dev not being built unless DEBUG_JS_MODULES is set, so while there is a slight perf overhead of dev, we won't need to have 2 copies in tree, and can still use the plugins like React.addons.TestUtils.
This isn't the best solution, but it'll work for now and give us more test coverage until we think of a way to do this, as this is the first time we're dealing with conditional envs in our builds with third party
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8678311 -
Flags: review?(jlong)
Comment 2•9 years ago
|
||
Comment on attachment 8678311 [details] [diff] [review]
1217979-react-dev.patch
Review of attachment 8678311 [details] [diff] [review]:
-----------------------------------------------------------------
+1, sucks that there are so many problems with it but we can figure it out later.
Attachment #8678311 -
Flags: review?(jlong) → review+
Comment 4•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•