Skip to content

Add back Update-List command#10922

Merged
anmenaga merged 4 commits intoPowerShell:masterfrom
SteveL-MSFT:update-list
Oct 30, 2019
Merged

Add back Update-List command#10922
anmenaga merged 4 commits intoPowerShell:masterfrom
SteveL-MSFT:update-list

Conversation

@SteveL-MSFT
Copy link
Copy Markdown
Member

@SteveL-MSFT SteveL-MSFT commented Oct 28, 2019

PR Summary

Re-enable Update-List cmdlet bringing parity with WinPS5.1. Added tests.

PR Checklist

@SteveL-MSFT
Copy link
Copy Markdown
Member Author

@sdwheeler do you have an updated fwlink for this cmdlet?

@sdwheeler
Copy link
Copy Markdown
Collaborator

@SteveL-MSFT https://go.microsoft.com/fwlink/?linkid=2109383

Copy link
Copy Markdown

@anmenaga anmenaga left a comment

Choose a reason for hiding this comment

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

Any info on why this cmdlet was removed originally?

@SteveL-MSFT
Copy link
Copy Markdown
Member Author

I don't know why it was originally removed since it should have been compatible since PSCore6. I don't find this cmdlet particularly useful (since you can just call the .NET API in PSScript), but a search on GitHub shows some scripts use it, so we should add it back for parity with WinPS.

@anmenaga anmenaga added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label Oct 30, 2019
@anmenaga anmenaga merged commit 0b12ea0 into PowerShell:master Oct 30, 2019
@anmenaga anmenaga added this to the 7.0.0-preview.6 milestone Oct 30, 2019
@SteveL-MSFT SteveL-MSFT deleted the update-list branch November 2, 2019 04:55
@ghost
Copy link
Copy Markdown

ghost commented Nov 21, 2019

🎉v7.0.0-preview.6 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants