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

Added Storybook template #318

Merged
merged 3 commits into from
Dec 15, 2019
Merged

Added Storybook template #318

merged 3 commits into from
Dec 15, 2019

Conversation

Carl-Foster
Copy link
Contributor

I gave this library a try to setup a design-library to be built with Storybook. Found it was more cumbersome than I thought but got it setup and felt it worthwhile to add a template here.
To enable further templates, I refactored how the template configs are defined but I'm not super happy with my implementation. Open to how that could be improved.
All feedback welcome

@Carl-Foster
Copy link
Contributor Author

@jaredpalmer is this PR fair enough? I'm happy to adjust it as I really appreciate this project.

@jaredpalmer
Copy link
Owner

@swyx? What u think? I’m fine with it?

@swyxio
Copy link
Collaborator

swyxio commented Dec 4, 2019

yup! fine with it too. am a fan of storybook

@swyxio
Copy link
Collaborator

swyxio commented Dec 4, 2019

@Carl-Foster mind adding the docs addon by default? its something i really want to encourage everyone uses

@Carl-Foster
Copy link
Contributor Author

@Carl-Foster mind adding the docs addon by default? its something i really want to encourage everyone uses

Sorry don't really understand what you mean by this.

@swyxio
Copy link
Collaborator

swyxio commented Dec 6, 2019

storybook addon docs is one of their biggest new things

@Carl-Foster
Copy link
Contributor Author

I unfortunately couldn't get the docs addon to work with Typescript seamlessly. I think if someone starts using the template can get it to work, then they can add it later.

@swyxio
Copy link
Collaborator

swyxio commented Dec 15, 2019

cool! thanks v much!

@swyxio swyxio merged commit c487377 into jaredpalmer:master Dec 15, 2019
sebald added a commit to sebald/tsdx that referenced this pull request Dec 19, 2019
* master: (26 commits)
  (deps/lint): upgrade @typescript-eslint to support ?. and ?? (jaredpalmer#377)
  (ci): add a lint job so PRs will require passing lint (jaredpalmer#378)
  (clean): remove .rts_cache_* from storybook gitignore (jaredpalmer#375)
  Add optional chaining and nullish coalescing operators support (jaredpalmer#370)
  Added Storybook template (jaredpalmer#318)
  (fix/ci): GitHub Actions should run on PRs as well
  (fix/format): formatting of jaredpalmer#366 didn't pass lint
  Add prepare script to generated project (jaredpalmer#334)
  default jest to watch mode when not in CI (jaredpalmer#366)
  (fix): respect tsconfig esModuleInterop flag (jaredpalmer#327)
  fix: minor typo
  update rollup deps and plugins
  update to ts 3.7
  Remove unnecessary yarn install command in GH action
  update README.md
  update README.md
  Use node_modules/.cache/... as cacheRoot (jaredpalmer#329)
  fix(lint): Only default to src test if they exist (jaredpalmer#344)
  Fix error when providing babel/preset-env without options (jaredpalmer#350)
  Replaced some sync methods for their async version
  ...
@swyxio
Copy link
Collaborator

swyxio commented Jan 24, 2020

thanks for the great PR! @all-contributors please add @Carl-Foster for ideas, docs, code, tests

@allcontributors
Copy link
Contributor

@sw-yx

I've put up a pull request to add @Carl-Foster! 🎉

@agilgur5 agilgur5 added the scope: templates Related to an init template, not necessarily to core (but could influence core) label Sep 21, 2020
@agilgur5 agilgur5 mentioned this pull request Nov 25, 2020
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: templates Related to an init template, not necessarily to core (but could influence core)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants