Skip to content

Addon-docs: Fix storybook MDX check#24696

Merged
ndelangen merged 4 commits into
nextfrom
shilman/24644-is-story-mdx
Dec 11, 2023
Merged

Addon-docs: Fix storybook MDX check#24696
ndelangen merged 4 commits into
nextfrom
shilman/24644-is-story-mdx

Conversation

@shilman
Copy link
Copy Markdown
Member

@shilman shilman commented Nov 3, 2023

Closes #24644

What I did

Fix check for .stories.mdx

Checklist for Contributors

Testing

The changes in this PR are covered in the following automated tests:

  • stories
  • unit tests
  • integration tests
  • end-to-end tests

Manual testing

See issue description.

🦋 Canary release

This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the @storybookjs/core team here.

core team members can create a canary release here or locally with gh workflow run --repo storybookjs/storybook canary-release-pr.yml --field pr=<PR_NUMBER>

@shilman shilman added bug addon: docs ci:normal Run our default set of CI jobs (choose this for most PRs). labels Nov 3, 2023
@bodograumann
Copy link
Copy Markdown
Contributor

Ah nice. This was easier than I thought.
I saw some files with -story.* and -stories.*, but looking more closely – those are not actual CSF files, but just part of the documentation of storybook.
🚀

@shilman shilman self-assigned this Nov 6, 2023
Comment thread code/lib/cli/src/automigrate/fixes/missing-babelrc.test.ts
@ndelangen ndelangen merged commit ed8a223 into next Dec 11, 2023
@ndelangen ndelangen deleted the shilman/24644-is-story-mdx branch December 11, 2023 15:36
@github-actions github-actions Bot mentioned this pull request Dec 11, 2023
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

addon: docs bug ci:normal Run our default set of CI jobs (choose this for most PRs).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Cannot use Story-of in History.mdx

5 participants