Skip to content

feat: type-check stories during Storybook build#876

Merged
agilgur5 merged 3 commits into
jaredpalmer:masterfrom
kylemh:patch-1
Sep 21, 2020
Merged

feat: type-check stories during Storybook build#876
agilgur5 merged 3 commits into
jaredpalmer:masterfrom
kylemh:patch-1

Conversation

@kylemh

@kylemh kylemh commented Sep 21, 2020

Copy link
Copy Markdown
Contributor

Resolves #668

@vercel

This comment has been minimized.

@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 changed the title Ensure stories are type-checked feat: type-check stories during Storybook build Sep 21, 2020

@agilgur5 agilgur5 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick patch @kylemh 🙂 That should be the last solvable Storybook issue for now, 😅

Some small comments changes below

Comment thread templates/react-with-storybook/.storybook/main.js
Comment thread templates/react-with-storybook/.storybook/main.js Outdated
Co-authored-by: Anton Gilgur <agilgur5@gmail.com>
@vercel vercel Bot temporarily deployed to Preview September 21, 2020 18:35 Inactive
Co-authored-by: Anton Gilgur <agilgur5@gmail.com>

@agilgur5 agilgur5 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@agilgur5 agilgur5 merged commit f1ddccb into jaredpalmer:master Sep 21, 2020
@kylemh kylemh deleted the patch-1 branch September 21, 2020 20:04
paul-vd pushed a commit to EezyQuote/tsdx that referenced this pull request Dec 1, 2020
- add `check: true` to Storybook TS config
  - which runs `fork-ts-checker-webpack-plugin` to check types
     asynchronously in a separate process during Storybook build

Co-authored-by: Anton Gilgur <agilgur5@gmail.com>
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.

stories dir is not being type checked during Storybook build

2 participants