Skip to content

Conversation

@daxian-dbw
Copy link
Member

@daxian-dbw daxian-dbw commented Aug 6, 2018

PR Summary

Update the 'HelpUri' for 'Get-ExperimentalFeature'.

PR Checklist

@daxian-dbw daxian-dbw requested a review from BrucePay as a code owner August 6, 2018 22:25
@daxian-dbw daxian-dbw removed the request for review from BrucePay August 6, 2018 22:25
@adityapatwardhan
Copy link
Member

The two changes are not related. I think we should not club them together in a single PR.

@daxian-dbw
Copy link
Member Author

@adityapatwardhan OK, I will separate them into two PRs.

@daxian-dbw daxian-dbw changed the title Update the 'HelpUri' for 'Get-ExperimentalFeature' and add back former repository maintainers Update the 'HelpUri' for 'Get-ExperimentalFeature' Aug 7, 2018
@daxian-dbw
Copy link
Member Author

@adityapatwardhan This PR has been updated to only contains the help uri change.

/// Implements Get-ExperimentalFeature cmdlet.
/// </summary>
[Cmdlet(VerbsCommon.Get, "ExperimentalFeature", HelpUri = "")]
[Cmdlet(VerbsCommon.Get, "ExperimentalFeature", HelpUri = "https://go.microsoft.com/fwlink/?linkid=2007351")]
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Will that test work when the URI currently points to nowhere?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done. csv file updated.

@adityapatwardhan adityapatwardhan self-assigned this Aug 7, 2018
@daxian-dbw daxian-dbw merged commit ca64bfb into PowerShell:master Aug 8, 2018
@daxian-dbw daxian-dbw deleted the help branch August 8, 2018 19:38
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