Skip to content

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