Skip to content

Release 0.7.2 #286

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
Sep 2, 2016
Merged

Release 0.7.2 #286

merged 4 commits into from
Sep 2, 2016

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Sep 2, 2016

Fixes for bugs that users reported in the 0.7.0 and 0.7.1 releases:

This change fixes an issue in PowerShellContext.AbortException which
causes it to throw a NullReferenceException when called after the session
has already been aborted.  This issue manifests sometimes when the
debug adapter finishes execution normally or when it terminates due to a
script parsing error.  It was initially reported in the following two
issues for the PowerShell VS Code extension:

PowerShell/vscode-powershell#243
PowerShell/vscode-powershell#273

The fix is to check whether the PowerShellContext.SessionState is Aborted
or Disposed before proceeding with the abort.
This change fixes an issue caused by the recent refactoring to use
PSScriptAnalyzer as a module rather than a C# library.  We forgot to pass
along the user-specified settings path to the -Settings parameter of the
Invoke-ScriptAnalyzer cmdlet.
@daviwil daviwil added this to the 0.7.2 milestone Sep 2, 2016
@daviwil daviwil changed the base branch from develop to master September 2, 2016 00:19
This change resolves an issue that appears when the user edits a script
that has dot-sourced script references which contain invalid characters.
The fix handles the exceptions that might be thrown in this case and
writes out the path details to the log file.

Related to PowerShell/vscode-powershell#274.
@daviwil
Copy link
Contributor Author

daviwil commented Sep 2, 2016

This one looks done to me. Any feedback @rkeithhill?

@rkeithhill
Copy link
Contributor

@daviwil LGTM

@daviwil daviwil merged commit 50ab29f into master Sep 2, 2016
@daviwil daviwil deleted the release/0.7.2 branch April 26, 2017 20:03
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