Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

debug-fb-www changes #1629

Conversation

trueadm
Copy link
Contributor

@trueadm trueadm commented Mar 21, 2018

Release notes: none

For debug-fb-ww we can recover from key and ref issues on ReactElements so we can evaluate more.

@@ -57,6 +57,10 @@ let prepackOptions = {
}
errorsCaptured.push(diag);
if (diag.severity !== "Warning") {
if (diag.errorCode === "PP0025") {
// recover from `unable to evaluate "key" and "ref" on a ReactElement due to an abstract config passed to createElement`,
Copy link
Contributor

Choose a reason for hiding this comment

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

Extra comma at the end?

Copy link
Contributor

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

Let's add a console log to the build script that reminds us we now assume ref/key aren't being spread. So we don't forget to come up with a solution for that (such as compiling to a different createElement in DEV that warns/forbids ref/key spreading).

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@trueadm is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants