-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
docs(types): fix type definitions #1678
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1678 +/- ##
=======================================
Coverage 99.75% 99.75%
=======================================
Files 141 141
Lines 2410 2410
=======================================
Hits 2404 2404
Misses 6 6
Continue to review full report at Codecov.
|
8a37929
to
40225a8
Compare
40225a8
to
1f1adb1
Compare
gulp/plugins/util/parseType.js
Outdated
return item | ||
}) | ||
|
||
const parseEnum = type => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this reduces the need for some of the logic in ComponentProps.js
:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've updated parser functions. renderEnums
method was simpilified, too 👍
Awesome, thanks much sir! This will also help with the speed a bit, there's a pretty bad performance bug I need to trace down in the component doc page. |
Released in |
Fixes #1563.
Before
After