-
Notifications
You must be signed in to change notification settings - Fork 4k
ServiceBus Command Help files #3422
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
Hi @v-Ajnava, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
TTYL, AZPRBOT; |
@v-Ajnava added |
updating the Service Bus PowerShell documentation
@cormacpayne - @sethmanheim and I just updated this PR. Can you start reviewing? |
@cormacpayne Hey Cormac, sorry for the nag. We'd love to get the help published for these cmdlets ASAP, since the module is already public. If you could review that'd be great. |
@jtaubensee @sethmanheim Hey John and Seth, thanks for updating the documentation. For these changes to appear in the cmdlets, you will need to update Microsoft.Azure.Commands.ServiceBus.dll-Help.xml, which can be done by using the Update-ServiceMarkdownHelp cmdlet that we have in our platyPSHelp module. Once that change is made, and the pull request is reviewed and merged, the changes will appear in the next release of Azure PowerShell on February 20th. Let me know if you guys have any other questions! |
@jtaubensee I can do this if you show me how to pull this branch locally. It didn't work too well yesterday... |
LGTM |
Description
This checklist is used to make sure that common guidelines for a pull request are followed. You can find a more complete discussion of PowerShell cmdlet best practices here.
General Guidelines