-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
Update reference to use packages from NuGet.org #7525
Conversation
using System.Management.Automation; | ||
using System.Security; | ||
using Microsoft.Management.Infrastructure; |
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.
Doesn't M
come before S
?
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.
All System* namespaces must be listed at top.
test/hosting/test_HostingBasic.cs
Outdated
|
||
using (System.Management.Automation.PowerShell ps = System.Management.Automation.PowerShell.Create()) | ||
{ | ||
string command = $"New-Item -ItemType SymbolicLink -Path {path} -Target {target}"; |
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.
Is this test because symlink depends on libpsl? Perhaps add a comment as Native
may be interpreted as a native command not from Microsoft.PowerShell.Native
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 will add a comment.
@SteveL-MSFT Addressed feedback, please have another look. |
restarted appveyor due to a PSVersion failure |
We need to update Microsoft.PowerShell.Native nuget package with adding new dependencies to pwrshcommon.cpp. Then update the csproj to take a dependency on the new version. |
* Update psrp and libpsl reference to Microsoft.PowerShell.Native * Keep the reference to powershell.myget.org in nuget.config for PSDesiredStateConfiguration and PowerShellHelpFiles * Update hosting tests * Update hosting.tests.csproj in preparation on next release with version 6.1.0-rc.1 * Update Microsoft.Management.Infrastructure to version 1.0.0
d4efe38
to
62446c9
Compare
PR Summary
PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
to the beginning of the title and remove the prefix when the PR is ready.[feature]
if the change is significant or affects feature tests