References to RunspaceConfiguration still present#5569
References to RunspaceConfiguration still present#5569daxian-dbw merged 1 commit intoPowerShell:masterfrom
Conversation
…PI documentation Updated the 2 unit tests accordingly with Runspaceconfiguration removal PowerShell#4942
|
|
SteveL-MSFT
left a comment
There was a problem hiding this comment.
I don't think we currently compile and run those csharp based tests
|
Indeed. I do not think you compile them ... otherwise they should fail for a couple of weeks. |
|
@Bhaal22 I think the changes to the test are correct, separate from this PR, someone should understand if that test code is still relevant if so, get it running. |
|
@SteveL-MSFT do you mean you would like to have 2 PRs? |
|
@Bhaal22 no, I think it's fine to have these changes as part of a single PR. I'm suggesting that investigating the utility of that test code is separate from this PR. |
RunspaceConfiguration still mentionned in RunspacePoolInternal ctor API documentation
Removed 2 obsolete test files
Those 2 files should even not compile.
#4942