-
Notifications
You must be signed in to change notification settings - Fork 367
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: [M3-7754] - fix bundle analyse script #10175
chore: [M3-7754] - fix bundle analyse script #10175
Conversation
Coverage Report: ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
packages/manager/package.json
Outdated
@@ -85,6 +85,7 @@ | |||
"start:ci": "yarn serve ./build -p 3000 -s --cors", | |||
"lint": "yarn run eslint . --ext .js,.ts,.tsx --quiet", | |||
"build": "node scripts/prebuild.mjs && vite build", | |||
"build:analyze": "npx vite-bundle-visualizer", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Optional: Use bunx
if we're feeling crazy 🤪
"build:analyze": "npx vite-bundle-visualizer", | |
"build:analyze": "bunx vite-bundle-visualizer", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you got it 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description 📝
I assume the script was broken since the vite update. This small PR makes it available again via vite-bunder-visualizer
Changes 🔄
Preview 📷
How to test 🧪
Reproduction steps
yarn build:analyze
Verification steps
(How to verify changes)
yarn build:analyze
As an Author I have considered 🤔
Check all that apply