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

Addon-docs: Fix source block tree shaking #15035

Merged
merged 1 commit into from
May 25, 2021
Merged

Conversation

shilman
Copy link
Member

@shilman shilman commented May 25, 2021

Issue: N/A

What I did

Source-block importing entire source-loader which is killing build times

How to test

Compare yarn storybook before & after

Mouse_Highlight_Overlay

@nx-cloud
Copy link

nx-cloud bot commented May 25, 2021

Nx Cloud Report

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

@yannbf yannbf left a comment

Choose a reason for hiding this comment

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

Great starting point, hopefully we can fix this in a better way later on!

@yannbf
Copy link
Member

yannbf commented May 25, 2021

Also kudos to @guilhermewaess for identifying the bug!

@shilman
Copy link
Member Author

shilman commented May 25, 2021

Yes, thank you @guilhermewaess !!! I'd seen those charts creep up so I knew there was something wrong, but since they actually start moving a few days before the change was introduced, it threw me off. You're a hero for figuring it out.

@shilman shilman merged commit 4220e92 into next May 25, 2021
@shilman shilman deleted the fix-doc-blocks-tree-shaking branch May 25, 2021 16:26
@guilhermewaess
Copy link
Contributor

Thanks, @shilman and @yannbf. I'm glad that I helped :D
I just tested the last version, and looks perfect 👍

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.

3 participants