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

Angular CLI update for new VS Code JS debugger #313

Merged
merged 2 commits into from
Aug 5, 2021

Conversation

rfuhrer
Copy link
Contributor

@rfuhrer rfuhrer commented Aug 4, 2021

Debugger for Chrome has been deprecated and the launch.json needs a couple tweaks to use the new build in debugger. Also threw in setting up Edge as well

Debugger for Chrome has been deprecated and the launch.json needs a couple tweaks to use the new build in debugger
@@ -39,15 +36,15 @@ Then click on the gear icon to configure a launch.json file, selecting **Chrome*

![add-chrome-debug](https://user-images.githubusercontent.com/2836367/27004175-77582668-4dca-11e7-9ce8-30ef3af64a36.png)

- Replace content of the generated launch.json with the following three configurations:
- Replace content of the generated launch.json with the following three configurations (use `pwa-msedge` for MS Edge):
Copy link
Member

Choose a reason for hiding this comment

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

The change to pwa-msedge/chrome isn't required, see microsoft/vscode-js-debug#1065

Copy link
Contributor Author

@rfuhrer rfuhrer Aug 5, 2021

Choose a reason for hiding this comment

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

Interesting, I was just changing that based on what VS Code was telling me:
image
image

Copy link
Member

Choose a reason for hiding this comment

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

The Debugger for Chrome is still needed to use type: chrome in the currently-released VS Code versions, but that will no longer be the case in 1.60.0 (which releases to stable early next month, and should arrive in Insiders today)

next version of VS Code will support chrome/msedge without needing pwa-
@connor4312 connor4312 merged commit 98b6d25 into microsoft:master Aug 5, 2021
@rfuhrer rfuhrer deleted the patch-1 branch August 5, 2021 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants