-
-
Notifications
You must be signed in to change notification settings - Fork 171
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
Support skip via story parameters #228
Conversation
<Welcome showApp={linkTo('Button')} /> | ||
)); | ||
storiesOf('Welcome', module) | ||
.lokiSkip('to Storybook', () => <Welcome showApp={linkTo('Button')} />) |
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.
So lokiSkip
will continue to exist and behave like add
?
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.
Yeah, i'll add warnings for now and remove it some time in the future.
warnedSkipDeprecation = true; | ||
console.warn( | ||
'[DEPRECATED] `.add.skip(...)` is deprecated. Please use `.lokiSkip(...)` instead.' | ||
'[DEPRECATED] `.add.skip(...)` and `.lokiSkip(...)` are deprecated. Please pass `{ loki: { skip: true } }` in the third parameter instead.' |
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.
Nice warning! 💯
Here too 😄 |
The `lokiSkip` method is deprecated. oblador/loki#228
The `lokiSkip` method is deprecated. oblador/loki#228
This PR moves the configuration of skipping individual stories to the story
parameter
field, deprecating thelokiSkip
method. Additionally it drops the deprecated use ofkind+story
combination and uses theid
introduced in SB 5.Fixes #176 #190 #198
This is a breaking change, dropping support for storybook prior to version 5.0 which was released about one year ago. It is probably possible to keep backwards support, but it would be a bit messy and would have to rely on heuristics.