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

Update launch.json to be useful for extension debugging #461

Merged
merged 1 commit into from
Jun 29, 2020

Conversation

viktomas
Copy link
Contributor

When I worked on debugging #432 (comment), I needed to make the extension work locally. This configuration change was very useful for running a local development version to package an extension.

],
"sourceMaps": true,
"outputCapture": "std",
"preLaunchTask": "compile"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This task isn't configured.

@joaomoreno
Copy link
Member

joaomoreno commented Jun 26, 2020

Isn't ls gonna fail if you don't provide a cwd? How is this better than the current state?

@joaomoreno joaomoreno self-assigned this Jun 26, 2020
@joaomoreno joaomoreno added the info-needed Issue requires more information from poster label Jun 26, 2020
@viktomas
Copy link
Contributor Author

Isn't ls gonna fail if you don't provide a cwd?

That's a good point. I got carried away with my usecase. Please see updated change @joaomoreno

@joaomoreno joaomoreno merged commit 820c014 into microsoft:master Jun 29, 2020
@joaomoreno joaomoreno added this to the June 2020 milestone Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info-needed Issue requires more information from poster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants