Skip to content

Improve detection of session changes when debugging ScriptBlocks #371

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 3 commits into from
Feb 17, 2017

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Feb 17, 2017

No description provided.

This change improves the logic in PushRunspaceIfSessionChanged to do a
better job of detecting local and remote session changes when debugging
ScriptBlocks and runspaces.
@daviwil daviwil added this to the 0.10.0 milestone Feb 17, 2017
This change enables the use of line-number breakpoints in DSC resources
when using a currently unreleased version of PowerShell DSC.  This
functionality may appear in a later version of PowerShell and DSC.
@daviwil daviwil force-pushed the daviwil/push-detect branch from 6405dd1 to 2cecbd5 Compare February 17, 2017 20:15
@daviwil
Copy link
Contributor Author

daviwil commented Feb 17, 2017

@rkeithhill @kapilmb Note that I've updated to .NET SDK RC4, you'll want to update to the latest VS 2017 RC build (check the Updates UI in VS) so that it can be built in Visual Studio

@daviwil daviwil merged commit 2aa8087 into develop Feb 17, 2017
@daviwil daviwil deleted the daviwil/push-detect branch February 17, 2017 20:23
@rkeithhill
Copy link
Contributor

Thanks. At this point, I'll probably wait for the RTM bits on March 7th. I've already hosed one machine with a VS 2017 RC (can't update to a newer version and can't fully uninstall the old version). :-(

@daviwil
Copy link
Contributor Author

daviwil commented Feb 17, 2017

I just realized that the RC4 bump won't have any effect on VS, if you're using the VS 2017 that supports RC3 everything will still build as it should.

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