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

Fix return type in onlyChild's JSDoc #6573

Merged
merged 1 commit into from Apr 21, 2016
Merged

Conversation

elas7
Copy link
Contributor

@elas7 elas7 commented Apr 21, 2016

The return type is a ReactElement instead of a ReactComponent.

The return type is a 'ReactElement' instead of a 'ReactComponent'.
@jimfb
Copy link
Contributor

jimfb commented Apr 21, 2016

We need to put an end to these one-line jsdoc fixes. Not sure what to do about the problem though. I don't want to reject these PRs because having wrong jsdocs is worse than having no jsdocs - it can be confusing as hell when the docs are lying to you.

Anyway, thanks @elas7! If you're fixing up jsdocs, it would be great if you could go through and get as many of them as possible squeezed into a single PR. But if you're just browsing the code and happened to see this one, great, thanks for fixing!

@jimfb jimfb merged commit 416f315 into facebook:master Apr 21, 2016
@elas7 elas7 deleted the onlychild-return branch April 21, 2016 23:07
@zpao zpao modified the milestones: 15.0.x, 15.0.2 Apr 22, 2016
zpao pushed a commit that referenced this pull request Apr 28, 2016
The return type is a 'ReactElement' instead of a 'ReactComponent'.
(cherry picked from commit 416f315)
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.

None yet

4 participants