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

Fixed regression introduced in #14563. #15005

Closed
wants to merge 1 commit into from

Conversation

coreyjv
Copy link
Contributor

@coreyjv coreyjv commented May 21, 2021

The regression prevented Axe tests from being run and instead the default snapshots were being used. This was due to a missing return statement that I overlooked in the original source because of the implicit return.

Additional context can be found here: https://github.com/storybookjs/storybook/pull/14563/files#r637109923

The regression prevented Axe tests from being run and instead the default snapshots were being used. This was due to a missing `return` statement that I overlooked in the original source because of the implicit return.
@nx-cloud
Copy link

nx-cloud bot commented May 21, 2021

Nx Cloud Report

CI ran the following commands for commit 1bd3938. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

Status Command
#000000 nx run-many --target=prepare --all --parallel --max-parallel=15

Sent with 💌 from NxCloud.

@shilman
Copy link
Member

shilman commented May 22, 2021

Looks like this was already fixed in #15006, closing. Thanks for following up!! 🙏

@shilman shilman closed this May 22, 2021
@coreyjv coreyjv deleted the regression-fix-from-14563 branch May 25, 2021 11:43
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

3 participants