-
Notifications
You must be signed in to change notification settings - Fork 357
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
feat: move components lib into this repo #1964
Conversation
f045e45
to
24ad13b
Compare
.github/workflows/deploy-ghpages.yml
Outdated
- name: Deploy docs to GitHub pages | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.SEMANTIC_RELEASE_GH }} | ||
run: npm run ghpages:deploy | ||
- name: Commit storybook for git hub doc site |
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.
one extra step here. we will need will need to checkout the branch gh-pages
before doing anything. we might also want to the folder name as storybook-static here.
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.
Thanks for the feedback. I added the storybook stuff into the github script. Please let me know if that looks alright, @owilliams320.
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 🚀 🥇 just had one comment
* Moved over from odubya/covalent-sigma. * Storybook running for components lib. * Fix dialog example page so that the buttons actually open the dialogs now.
24ad13b
to
d7e6896
Compare
🎉 This PR is included in version 4.6.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Description
Test Steps
npm run storybook
General Tests for Every PR
npm run lint
passes.npm test
passes and code coverage is not lower.Screenshots or link to StackBlitz/Plunker