Skip to content

WIP: Test cherry picks #882

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

Closed

Conversation

TylerLeonhardt
Copy link
Member

I am waiting on #881 to be merged in before merging this in.

This should probably go in before #880

TylerLeonhardt and others added 10 commits January 23, 2019 08:44
* Prep 1.11 release with Changelog

* spaces
* Added `AsNewFile` switch, plus, if a file is not open, it creates a new one

* Update module/PowerShellEditorServices/Commands/Public/Out-CurrentFile.ps1

Co-Authored-By: dfinke <[email protected]>
* Return the start line number for Describe block

Supports PR 1776 in vscode-powershell

* Null check pesterSymbol.ScriptRegion

* Update comment to cause new build to kick off

* Whitespace change to kick off a new build
* Add attach to local runspace.

* Remove comment.

* Use single runspaceId property.

* Remove unnecessary processId check.

* Factored if.

* Update src/PowerShellEditorServices.Protocol/Server/DebugAdapter.cs

It's neat that we can just commit a suggestion.

Co-Authored-By: adamdriscoll <[email protected]>

* Use runspace rather than dynamic.

* Accept process ID for GetRunspaceRequest.

* Clean up.
@@ -12,7 +12,7 @@
RootModule = 'PowerShellEditorServices.psm1'

# Version number of this module.
ModuleVersion = '1.10.3'
ModuleVersion = '1.11.0'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will get bumped to 1.11.1 before the release, right?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah it will be 1.12.0 because weve got features!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even better.

Copy link
Contributor

@rkeithhill rkeithhill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM modulo the merge conflicts being fixed.

@TylerLeonhardt
Copy link
Member Author

Yeah this will be closed and I will do it to the branch directly so we keep the full history.

@TylerLeonhardt
Copy link
Member Author

this should go into legacy

@TylerLeonhardt TylerLeonhardt deleted the test-cherry-picks branch September 16, 2019 16:10
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.

6 participants