Skip to content

Release 0.7.1 #277

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 5 commits into from
Aug 24, 2016
Merged

Release 0.7.1 #277

merged 5 commits into from
Aug 24, 2016

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Aug 23, 2016


This change is Reviewable

@daviwil
Copy link
Contributor Author

daviwil commented Aug 23, 2016

Hmmm, I think I incorrectly referenced one of those issues...

This change resolves some compatibility issues with PowerShell v3 and v4.
The new PowerShellEditorServices.psm1 file was using the ::new()
constructor syntax that is only available in PowerShell v5 and higher.
Also, the AstVisitor2 class was causing issues with Add-Type when being
loaded into the older versions of PowerShell.

Issue #276 tracks the need for bringing AstVisitor2 usage back after a
more appropriate multi-version support strategy is designed.

Related to PowerShell/vscode-powershell#248
This change restores the default set of rules that we previously used in
our PSScriptAnalyzer integration.  This list got removed inadvertently
when we changed our design to consume PSScriptAnalyzer as a module rather
than a C# API.

Related to PowerShell/vscode-powershell#246
This change causes the "Auto" variable scope to be marked as not
'expensive' in the VS Code debug protocol so that it is always expanded by
default in VS Code's debug client UI.
This change temporarily removes the OS and process information gathering
from the CoreCLR version of PowerShell Editor Services because they cause
assembly loading errors.  This should be resolved by the upcoming conversion
to the dotnet cli tools.

Future resolution tracked by #278.
@daviwil daviwil merged commit 30f246b into develop Aug 24, 2016
@daviwil daviwil deleted the release/0.7.1 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.

2 participants