-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
docs: add shamefully hoist mention for pnpm users on example launch.json #26447
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
Conversation
|
You may need to update the config below with a path to your web browser. For more information, visit the [VS Code documentation about debug configuration](https://go.microsoft.com/fwlink/?linkid=830387). | ||
|
||
::important | ||
Make sure you have shamefully-hoist enabled if you use pnpm to properly start your server in debug mode with below configuration. |
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.
Wouldn't it be better to suggest adding nuxi
to devDependencies?
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.
Ah yes thats a cleaner solution. I didn't come up with that before.
Would:
Make sure you have nuxi installed as Devdependency if you use pnpm to properly start your server in debug mode with below configuration.
suffice?
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.
That sounds good!
If you use
pnpm
, you will need to havenuxi
installed as a devDependency for the configuration below to work.
π Linked issue
β Type of change
π Description
π Checklist