Skip to content

Conversation

andyleejordan
Copy link
Member

Thanks to a brilliant idea from Patrick, this simplifies how we choose which commands to execute through PowerShell's debugger API, and also fixes an issue where a user's custom prompt function wasn't aware that it was in the debugger.

This fixes PowerShell/vscode-powershell#3980.

Thanks to a brilliant idea from Patrick, this simplifies how we choose
which commands to execute through PowerShell's debugger API, and also
fixes an issue where a user's custom prompt function wasn't aware that
it was in the debugger.
@andyleejordan andyleejordan requested a review from a team May 17, 2022 19:41
@andyleejordan andyleejordan enabled auto-merge (squash) May 17, 2022 19:42
Copy link
Collaborator

@SeeminglyScience SeeminglyScience left a comment

Choose a reason for hiding this comment

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

One thing needs to change real quick otherwise LGTM

…chronousPowerShellTask.cs

Co-authored-by: Patrick Meinecke <[email protected]>
@andyleejordan
Copy link
Member Author

Good catch! Committed.

@SeeminglyScience SeeminglyScience self-requested a review May 17, 2022 19:46
Copy link
Collaborator

@SeeminglyScience SeeminglyScience left a comment

Choose a reason for hiding this comment

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

LGTM!

@andyleejordan andyleejordan merged commit a4ea69a into master May 17, 2022
@andyleejordan andyleejordan deleted the andschwa/debug-prompt branch May 17, 2022 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Custom prompt function no longer sees PSDebugContext
2 participants