-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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 breakpoint debugging guide #12212
docs: add breakpoint debugging guide #12212
Conversation
|
"version": "0.2.0", | ||
"configurations": [ | ||
{ | ||
"command": "npm start", |
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.
I think it'd be helpful to make this npm run dev
to match the text above and real projects so that the user can copy/paste this example
Thanks, Ben! I also added a link to Svelte Inspector for good measure. |
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.
Proposed changes to the svelte.dev site need to target the svelte-4
branch, not main
.
Apologies, I can't read. This is |
"configurations": [ | ||
{ | ||
"command": "npm run dev", | ||
"name": "Run npm start", |
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.
"name": "Run npm start", | |
"name": "Run npm dev", |
Can we change the page prefix from |
|
||
## Visual Studio Code | ||
|
||
With the build-in debug terminal, you can set up breakpoints in source files within VSCode. |
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.
With the build-in debug terminal, you can set up breakpoints in source files within VSCode. | |
With the built-in debug terminal, you can set up breakpoints in source files within VSCode. |
@theetrain I think there are a couple comments outstanding on this PR. If you're able to address them I'd love to get this in. We'll be restructuring the docs and closing any open PRs, so I'd like to get it in before that happens |
5585533
to
187f3db
Compare
Closes #7781, relates to #1144
While testing with the Node.js inspector, I tried to enable server-side breakpoint debugging, but was not successful. The sourcemaps are being generated, but only client-side code connect to chrome devtools. VSCode breakpoint debugging works well.
I'm hoping there could be a way to debug server-side code within chrome devtools as well, which would help developers who aren't using VSCode, but I'm unsure whose domain this falls in: SvelteKit, Vite, or something else.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
Tests
pnpm test
and lint the project withpnpm lint
andpnpm check
Changesets
pnpm changeset
and following the prompts. Changesets that add features should beminor
and those that fix bugs should bepatch
. Please prefix changeset messages withfeat:
,fix:
, orchore:
.Edits