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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build: fix test CI job #12989

Closed
wants to merge 1 commit into from
Closed

Build: fix test CI job #12989

wants to merge 1 commit into from

Conversation

gaetanmaisse
Copy link
Member

@gaetanmaisse gaetanmaisse commented Nov 2, 2020

Issue: test CI job is broken on next https://app.circleci.com/pipelines/github/storybookjs/storybook/14448/workflows/2be541c1-55e0-4328-8dda-326424fe3aa4

When #12972 will be merged the remaining errors will be related to snapshots of MDX tests.

What I did

Update snapshots of MDX tests after the update of @mdx-js/* packages made in fe2a9e2

Snapshots updates look to match the release note:
https://github.com/mdx-js/mdx/releases/tag/v1.6.16

Only add makeShortcode function if it's needed

How to test

  • CI should be 馃煝 (except Yarn 2)

Snapshots updates look to match the release note:
https://github.com/mdx-js/mdx/releases/tag/v1.6.16

> Only add makeShortcode function if it's needed

Update was made in fe2a9e2
@gaetanmaisse gaetanmaisse added the maintenance User-facing maintenance tasks label Nov 2, 2020
@gaetanmaisse
Copy link
Member Author

Already fixed in next

@gaetanmaisse gaetanmaisse deleted the tech/fix-tests branch November 3, 2020 07:12
@shilman
Copy link
Member

shilman commented Nov 3, 2020

Oops @gaetanmaisse sorry I missed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants