-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
docs: add debugging section #6857
base: main
Are you sure you want to change the base?
Conversation
|
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.
It looks good to me 👏 thanks @czuma
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
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.
Nice, but isn't it easier to use npx vite-bundle-visualizer
? Could even be a default script in the starter
THAT'S GREAT! i tested and works amazing. Maybe we can split this in 2 parts, 1 is document this command instead of my PR, and 2nd part is to add to starter like you say. Let me work on that. |
@czuma I think you can do both things in this PR. Looking forward to our new debugging sections in the docs! 😃 |
right, i left this unfinished, will work on this this week. |
What is it?
Description
Checklist:
pnpm change
and documented my changes