Skip to content

Fix tests in Examples folder to all pass. #434

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

Conversation

rkeithhill
Copy link
Contributor

Three of them started to fail when I added another psd1 file to the folder (for the PSSA settings).

Three of them started to fail when I added another psd1 file to the folder (for the PSSA settings).
@rkeithhill rkeithhill merged commit 1ce8b35 into PowerShell:master Jan 14, 2017
@rkeithhill rkeithhill deleted the rkeihthill/fix-example-tests branch January 14, 2017 17:10
@rkeithhill rkeithhill changed the title Fix tests in Exampls folder to all pass. Fix tests in Examples folder to all pass. Jan 14, 2017
daviwil added a commit to daviwil/vscode-powershell that referenced this pull request May 31, 2017
This change fixes an issue which causes the language server to crash
when running a registered $psEditor command from within an untitled
file.  This happens because the workspace code in the language server
expects to receive Uri paths instead of local file paths when referring
to an editor buffer.  The fix is to send the untitled file's Uri instead
of its non-existent file path.

Resolves PowerShell#810.
Resolves PowerShell/PowerShellEditorServices PowerShell#434.
daviwil added a commit that referenced this pull request May 31, 2017
This change fixes an issue which causes the language server to crash
when running a registered $psEditor command from within an untitled
file.  This happens because the workspace code in the language server
expects to receive Uri paths instead of local file paths when referring
to an editor buffer.  The fix is to send the untitled file's Uri instead
of its non-existent file path.

Resolves #810.
Resolves PowerShell/PowerShellEditorServices #434.
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