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

Core: Deprecate configure and clearDecorators #11431

Merged
merged 5 commits into from
Jul 6, 2020
Merged

Conversation

tmeasday
Copy link
Member

@tmeasday tmeasday commented Jul 6, 2020

Issue: for #10005

What I did

  • Deprecated configure
  • Stopped HMR for configure to be consistent with main.js:stories

How to test

Try adding a configure() call to preview.js in official SB

This avoids us having to solve #10005 right now and is consistent with the stories entry.
(So it is easier to have a trailing arg)
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

Awesome!!! ❤️

@@ -1,6 +1,3 @@
import { configure } from '@storybook/{{framework}}';

// eslint-disable-next-line no-underscore-dangle
module._StorybookPreserveDecorators = true;
Copy link
Member Author

Choose a reason for hiding this comment

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

This wasn't doing anything any more anyway.

@shilman shilman changed the title Deprecate configure Core: Deprecate configure and clearDecorators Jul 6, 2020
@shilman shilman merged commit 16a85b9 into next Jul 6, 2020
@shilman shilman deleted the 10005-deprecate-configure branch July 6, 2020 13:48
@tmeasday tmeasday mentioned this pull request Jul 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants