Improve debugging support #30
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This change introduces the new OutputEvent which allows the the language and debugging services to send script output text to the client to be displayed to the user. Currently this is used in VS Code's Debug Console but could be used in other contexts as well. This change also enables the user to evaluate PowerShell commands while stopped in the debugger to aid in debugging. In the future this will be expanded to allow commands to be executed at any time.
This change adds the dollar sign '$' character to top-level variables that are retrieved using the Get-Variables command via the DebugService. This character is used to refer to variables in PowerShell scripts so we need to add it to variable names that are returned from the debugging service for the sake of consistency.
|
Failed due to intermittent test failure issue #27, but actual code that was changed all has passing tests. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See individual commit messages for more details.