-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
References to RunspaceConfiguration still present #5569
Conversation
…PI documentation Updated the 2 unit tests accordingly with Runspaceconfiguration removal PowerShell#4942
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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