-
Notifications
You must be signed in to change notification settings - Fork 33
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
#trivial Add storybook to the master branch #1314
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/styleguide-core/styleguide/i31592dz6 |
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.
How do you feel in adding v6.x instead of v5.x? 🤔
@matheusps Already migrated to v6.x 😄 |
97e3606
to
a2f8e49
Compare
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 4f9b71f:
|
Oh, I see. So, why knoobs and not controls ? |
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.
Just the question about controls. The rest, LGTM
└── button.stories.tsx | ||
``` | ||
|
||
#### Conventions |
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.
✨
a2f8e49
to
e74bcd0
Compare
e74bcd0
to
e715e73
Compare
@matheusps Thanks for the reminder! I forgot about the knobs and actions 😄 could you check again? |
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.
LGTM ✨
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.
Looks awesome
e715e73
to
c7606ce
Compare
c7606ce
to
4f9b71f
Compare
What is the purpose of this pull request?
This is a previously work done to the branch
features
, but since this branch has been discontinued I'm moving to themaster
.What problem is this solving?
We need an environment with hot-loading and isolated components builds to help us develop our components.
How should this be manually tested?
yarn && yarn storybook
Visit http://localhost:6006/
Screenshots or example usage
Types of changes