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 multiple components in the same file issues #17

Merged
merged 3 commits into from
Nov 1, 2016
Merged

Conversation

madushan1000
Copy link
Contributor

Addresses issue #14. More testing is needed.

const types = [
'ExportDeclaration',
'ExportNamedDeclaration',
'ExportDefaultDeclaration'
Copy link
Member

Choose a reason for hiding this comment

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

I assume we only need to check for the default declaration. Isn't it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

React docgen will process the file if there is only one named declaration(multiple named declarations fail). So we have to check for that too.

Copy link
Member

Choose a reason for hiding this comment

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

Got it.

@arunoda arunoda merged commit cb7d1e2 into master Nov 1, 2016
@arunoda arunoda deleted the multiclass-issues branch November 1, 2016 08:39
@arunoda arunoda changed the title multiple components in the same file issues Fix multiple components in the same file issues Nov 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants