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

fix: react build-storybook hang process #15709

Closed
wants to merge 1 commit into from
Closed

fix: react build-storybook hang process #15709

wants to merge 1 commit into from

Conversation

JayBee007
Copy link

@JayBee007 JayBee007 commented Jul 29, 2021

Issue: build-storybook output static files but the process never finishes

What I did

Copied the fix for angular's unfinished build-storybook process from the PR: #15483

How to test

Try to build storybook with a fully controlled custom webpack config on CI/CI

@nx-cloud
Copy link

nx-cloud bot commented Jul 29, 2021

Nx Cloud Report

CI ran the following commands for commit a08aaee. 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.

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.

Thanks for the PR @JayBee007! However, I think this change is redundant with the code that you're wrapping:

https://github.com/storybookjs/storybook/blob/next/lib/core-server/src/build-static.ts#L147-L149

Closing this for now. Please let me know if I'm missing something! 🙏

@shilman shilman closed this Aug 2, 2021
@JayBee007
Copy link
Author

Got it! Thanks for the correction.

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

Successfully merging this pull request may close these issues.

None yet

2 participants