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 debug-visual-studio-code.md to specify app exe name #3240

Merged
merged 5 commits into from
Jul 30, 2024

Conversation

champnic
Copy link
Contributor

@champnic champnic commented Jul 29, 2024

Rendered article section for review:

Devs should specify a specific app exe name in the reg key when setting up debugging, rather than using '*' which will apply to all WebView2 apps.

AB#52889881

Devs should specify a specific app exe name in the reg key when setting up debugging.
Copy link
Contributor

@mikehoffms mikehoffms left a comment

Choose a reason for hiding this comment

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

Approved PR after pushing commits for Writer/Editor pass

Copy link
Contributor Author

@champnic champnic left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

Learn Build status updates of commit 28dfdda:

✅ Validation status: passed

File Status Preview URL Details
microsoft-edge/webview2/how-to/debug-visual-studio-code-images/set-debugging-port-registry-key.png ✅Succeeded View
microsoft-edge/webview2/how-to/debug-visual-studio-code-images/set-debugging-port.png ✅Succeeded View
microsoft-edge/webview2/how-to/debug-visual-studio-code.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@victorhuangwq victorhuangwq left a comment

Choose a reason for hiding this comment

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

LGTM

@Reezaali Reezaali merged commit ee92916 into main Jul 30, 2024
2 checks passed
@Reezaali Reezaali deleted the champnic-52885780 branch July 30, 2024 16:50
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.

4 participants