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

♻️ Storybook in Component Story Format #30329

Merged
merged 1 commit into from
Sep 22, 2020

Conversation

alanorozco
Copy link
Member

@alanorozco alanorozco commented Sep 22, 2020

From Storybook's docs:

In Storybook 5.2 we introduced a simpler and more portable Component Story Format, and all future tools and infrastructure will be oriented towards CSF. Therefore, we recommend migrating your stories out of storiesOf API, and even provide automated tools to do this.

AMP samples were using storiesOf(), but Preact samples already used Component Story Format.

Used migration tool mentioned.

@google-cla google-cla bot added the cla: yes label Sep 22, 2020
alanorozco added a commit to alanorozco/storybook-addon-amp that referenced this pull request Sep 22, 2020
From [Storybook's docs](https://github.com/storybookjs/storybook/blob/next/lib/core/docs/storiesOf.md):

> In Storybook 5.2 we introduced a simpler and more portable [Component Story Format](https://storybook.js.org/docs/react/api/csf), and all future tools and infrastructure will be oriented towards CSF. Therefore, we recommend migrating your stories out of `storiesOf` API, and even provide [automated tools to do this](https://github.com/storybookjs/storybook/blob/next/lib/core/docs/storiesOf.md#component-story-format-migration).

Also: ampproject/amphtml#30329
@alanorozco alanorozco merged commit b914854 into ampproject:master Sep 22, 2020
@alanorozco alanorozco deleted the no-storiesof branch September 22, 2020 18:53
Copy link
Contributor

@krdwan krdwan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for doing this!

Copy link
Contributor

@krdwan krdwan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for doing this!

@alanorozco alanorozco mentioned this pull request Oct 8, 2020
6 tasks
ed-bird pushed a commit to ed-bird/amphtml that referenced this pull request Dec 10, 2020
From [Storybook's docs](https://github.com/storybookjs/storybook/blob/next/lib/core/docs/storiesOf.md):

> In Storybook 5.2 we introduced a simpler and more portable [Component Story Format](https://storybook.js.org/docs/react/api/csf), and all future tools and infrastructure will be oriented towards CSF. Therefore, we recommend migrating your stories out of `storiesOf` API, and even provide [automated tools to do this](https://github.com/storybookjs/storybook/blob/next/lib/core/docs/storiesOf.md#component-story-format-migration).

AMP samples were using `storiesOf()`, but Preact samples already used Component Story Format.

Used migration tool mentioned.
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.

None yet

4 participants