-
-
Notifications
You must be signed in to change notification settings - Fork 9.7k
Fix to README add-ons section #3079
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
Conversation
Add unmentioned addons Alphabetize addons
@@ -3,7 +3,7 @@ | |||
[](https://circleci.com/gh/storybooks/storybook) | |||
[](https://www.codefactor.io/repository/github/storybooks/storybook) | |||
[](https://snyk.io/test/github/storybooks/storybook/8f36abfd6697e58cd76df3526b52e4b9dc894847) | |||
[](https://bettercodehub.com/results/storybooks/storybook) [](https://codecov.io/gh/storybooks/storybook) |
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.
This was just my editor removing trailing whitespace.
Codecov Report
@@ Coverage Diff @@
## master #3079 +/- ##
=======================================
Coverage 36.08% 36.08%
=======================================
Files 438 438
Lines 9447 9447
Branches 882 905 +23
=======================================
Hits 3409 3409
+ Misses 5472 5457 -15
- Partials 566 581 +15
Continue to review full report at Codecov.
|
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.
Cool, thanks
Issue:
There was a dead link in the main README file to a non-existent comments add-on.
What I did
I removed the dead link. While I was in there, I added links and short blurbs for the add-ons that weren't listed. I also alphabetized the list of add-ons.
How to test
N/A
Is this testable with jest or storyshots?
No
Does this need a new example in the kitchen sink apps?
No
Does this need an update to the documentation?
This is an update to documentation.
If your answer is yes to any of these, please make sure to include it in your PR.