Skip to content

A few minor fixes for the 0.4.0 release #135

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

Merged
merged 4 commits into from
Feb 5, 2016
Merged

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Feb 5, 2016

No description provided.

This change adds a /logLevel parameter to the host process so that log
output verbosity can be controlled by an editor setting.

Resolves #132
This change removes the
Microsoft.PowerShell.EditorServices.DebugAdapter.cmd files which were used
to wrap invocation of the host process to pass the /debugAdapter
parameter.  This was needed because VS Code did not previously have a way
to pass command line parameters to debug adapters at startup.

Resolves #131
@adamdriscoll
Copy link
Contributor

Based on the amount of GitHub notifications I'm getting I would bet there is a release coming soon. ;)

@daviwil
Copy link
Contributor Author

daviwil commented Feb 5, 2016

Yeeeeep ;) Shooting for Monday! We'll see when the VS Code team drops 0.10.7.

This change keeps the debug adapter from printing the prompt until the
user enters a command.  This will allow breakpoints to be hit without the
prompt being shown every time.  The change also trims the ending '>'
character off of the end of the prompt to reduce user confusion about
where commands should be typed.
This change writes out the host process ID when the process starts up so
that it's easier to determine which process to attach a debugger to.
@adamdriscoll
Copy link
Contributor

🤘

@daviwil daviwil force-pushed the daviwil/minor-fixes branch from a3e0cff to dba7e97 Compare February 5, 2016 04:33
@daviwil
Copy link
Contributor Author

daviwil commented Feb 5, 2016

Nice, that emoji is awesome.

daviwil added a commit that referenced this pull request Feb 5, 2016
A few minor fixes for the 0.4.0 release
@daviwil daviwil merged commit 5c69c7d into master Feb 5, 2016
@daviwil daviwil deleted the daviwil/minor-fixes branch February 16, 2016 04:53
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.

3 participants