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

chore(v2): fix build size bot monitoring of js/css assets #4156

Merged
merged 2 commits into from
Feb 2, 2021

Conversation

slorber
Copy link
Collaborator

@slorber slorber commented Feb 2, 2021

Motivation

Build size bot got broken by #3998

Note: we should likely figure out a way to output better chunk names as we have large bundles named "1" and "3" and even worst namings.

Probably worth investing in this after the Webpack 5 migration

@slorber slorber added the pr: bug fix This PR fixes a bug in a past release. label Feb 2, 2021
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Feb 2, 2021
@netlify
Copy link

netlify bot commented Feb 2, 2021

[V1] Deploy preview success

Built with commit b25e57f

https://deploy-preview-4156--docusaurus-1.netlify.app

@netlify
Copy link

netlify bot commented Feb 2, 2021

Deploy preview for docusaurus-2 ready!

Built with commit b25e57f

https://deploy-preview-4156--docusaurus-2.netlify.app

@github-actions
Copy link

github-actions bot commented Feb 2, 2021

Size Change: +2 B (0%)

Total Size: 28.8 kB

ℹ️ View Unchanged
Filename Size Change
website/build/blog/2017/12/14/introducing-docusaurus/index.html 21.7 kB +1 B (0%)
website/build/docs/introduction/index.html 180 B 0 B
website/build/index.html 6.89 kB +1 B (0%)

compressed-size-action

@github-actions
Copy link

github-actions bot commented Feb 2, 2021

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 75
🟢 Accessibility 99
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 95

Lighthouse ran on https://deploy-preview-4156--docusaurus-2.netlify.app/classic/

@slorber
Copy link
Collaborator Author

slorber commented Feb 2, 2021

I think merging is needed to test it (due to using pull_request_target security in Github CI)

@slorber slorber added pr: maintenance This PR does not produce any behavior differences to end users when upgrading. and removed pr: bug fix This PR fixes a bug in a past release. labels Feb 2, 2021
@slorber slorber merged commit fe6b841 into master Feb 2, 2021
@slorber slorber changed the title fix(v2): fix build size bot monitoring of js/css assets chore(v2): fix build size bot monitoring of js/css assets Feb 2, 2021
@lex111 lex111 added this to the v2.0.0-alpha.71 milestone Mar 1, 2021
@slorber slorber deleted the slorber/fix-build-size-bot branch August 17, 2021 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: maintenance This PR does not produce any behavior differences to end users when upgrading.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants