-
Notifications
You must be signed in to change notification settings - Fork 4k
Clu test project and updates to clu-getstart.md #1680
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 @BurtBiel, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
TTYL, AZPRBOT; |
@@ -1,35 +0,0 @@ | |||
using System; |
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.
This file was unused, so deleting it for now
Tests pass in VS and with dnx test |
@@ -25,7 +25,7 @@ CLUPackages require some additional files to direct generation of indexing, and | |||
| ------------- |:-------------:| | |||
| CommandAssemblies | File name of cmdlets assembly(ies) | | |||
| NounPrefix | ‘AzureRm’ The part of the cmdlet noun to remove in clu commands| | |||
| NounFirst | if true, the verb comes at the end of the command (e.g. az resource get)| | |||
| NounFirst | if true, the verb comes at the end of the command (e.g. az resourcemanager resource ls)| |
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.
@BurtBiel this should be az resource ls
Addressed core review feedback |
It looks like one of our dependencies was removed, so I updated the Microsoft.CLU.Run |
@markcowl addressed code review feedback and updated a nugget package resource to get the build passing |
Clu test project and updates to clu-getstart.md
Clu test project and updates to clu-getstart.md