-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Chore: (Docs) Updates Storybook configuration docs for TS and flags. #17358
Conversation
☁️ Nx Cloud ReportCI ran the following commands for commit 5fe5b45. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
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.
small change otherwise LGTM!
import type { StorybookConfig } from '@storybook/core-common'; | ||
|
||
const BaseConfig: StorybookConfig = { | ||
stories: [], |
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.
Can we make this into a realistic file? In the current form it doesn't seem to be very useful. Also baseConfig
since it's a variable?
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.
Indeed we can, and I've updated it with the latest commit. Let me know if you're ok with it, or any other feedback. Thanks for taking the time to review it @shilman !
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!! 💯
…onfig_overview Chore: (Docs) Updates Storybook configuration docs for TS and flags.
Adding |
With this pull request, the documentation is updated to include TS support and available feature flags.
What was done:
Closes #16079 and #15629
@shilman would appreciate some feedback on this, and please let me know if I forgot any feature flags or something of that nature. Thanks in advance.