Skip to content
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

Warn if childContextType is defined on SFC #6933

Merged
merged 1 commit into from
Jun 8, 2016

Conversation

aweary
Copy link
Contributor

@aweary aweary commented Jun 1, 2016

Resolves #6926

Per the discussion there, this only warns for childContextTypes for now.

reactEurope-hackathon

@ghost
Copy link

ghost commented Jun 1, 2016

@aweary updated the pull request.

Add console.error message content check

Use appropriate name in warning/test
@ghost
Copy link

ghost commented Jun 1, 2016

@aweary updated the pull request.

@jimfb
Copy link
Contributor

jimfb commented Jun 8, 2016

Everything is technically public API. cc @facebook/react-core

I like it, and Ben gave the conceptual signoff, so I'm shipping it. Also over a week old/reviewed/green.

@jimfb jimfb merged commit c47830d into facebook:master Jun 8, 2016
@zpao zpao added this to the 15-next milestone Jun 8, 2016
zpao pushed a commit that referenced this pull request Jun 14, 2016
Add console.error message content check

Use appropriate name in warning/test
(cherry picked from commit c47830d)
@zpao zpao modified the milestones: 15-next, 15.2.0 Jun 14, 2016
aweary referenced this pull request Jul 5, 2016
(cherry picked from commit 74c29b3 and  bc1d59e)
gaearon added a commit that referenced this pull request Jul 5, 2016
zpao pushed a commit that referenced this pull request Jul 8, 2016
usmanajmal pushed a commit to usmanajmal/react that referenced this pull request Jul 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants