-
Notifications
You must be signed in to change notification settings - Fork 102
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
#712: Azure CLI integration #741
Conversation
add configuration file in azurecli.asciidoc
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.
Good work!
I have added some comments that you are welcome to implement or even discuss if you see something differently.
Please still agree with the Constribution Guildlines.
Please also check the shellchecker. There is one error.
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.
@alfeilex Thanks for this PR. You have done a very good job and it is more or less ready to be merged. Some of my comments are rather picky and are just nice-to-have hints for you. I would have merged right away but the last 2 comments seem to require a little improvement.
@hohwille I have applied the changes. The code is now more compact. Thank you for the comments. |
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.
@alfeilex Excellent. 👍 Thanks for your changes. Now everything looks perfect to me.
Ready for merge.
Azure CLI can be installed in Windows via Command
devon az