-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Update the 'HelpUri' for 'Get-ExperimentalFeature' #7466
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
Conversation
|
The two changes are not related. I think we should not club them together in a single PR. |
|
@adityapatwardhan OK, I will separate them into two PRs. |
|
@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")] |
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.
Please add a test for this by adding an entry at: https://github.com/PowerShell/PowerShell/blob/master/test/powershell/engine/Help/assets/HelpURI/V3Cmdlets.csv
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.
Will that test work when the URI currently points to nowhere?
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.
Done. csv file updated.
PR Summary
Update the 'HelpUri' for 'Get-ExperimentalFeature'.
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files 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