Skip to content
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

Merged
merged 19 commits into from
May 31, 2022
Merged

#712: Azure CLI integration #741

merged 19 commits into from
May 31, 2022

Conversation

alfeilex
Copy link
Member

@alfeilex alfeilex commented May 12, 2022

Azure CLI can be installed in Windows via Command devon az

@CLAassistant
Copy link

CLAassistant commented May 12, 2022

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added bash related to bash shell or scripts commandlet related to commandlets (scripts/command/*) documentation related to documentation (AsciiDoc) scripts related to shell scripts (bash and CMD) labels May 12, 2022
@alfeilex alfeilex linked an issue May 12, 2022 that may be closed by this pull request
12 tasks
@alfeilex alfeilex mentioned this pull request May 12, 2022
12 tasks
Copy link
Member

@tobka777 tobka777 left a 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.

scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
documentation/az.asciidoc Outdated Show resolved Hide resolved
documentation/az.asciidoc Outdated Show resolved Hide resolved
@hohwille hohwille changed the title Feature/azurecli #712: Azure CLI integration (#741) May 16, 2022
@hohwille hohwille changed the title #712: Azure CLI integration (#741) #712: Azure CLI integration May 16, 2022
@alfeilex alfeilex requested a review from hohwille May 19, 2022 08:22
Copy link
Member

@hohwille hohwille left a 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.

documentation/az.asciidoc Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Outdated Show resolved Hide resolved
scripts/src/main/resources/scripts/command/az Show resolved Hide resolved
@alfeilex
Copy link
Member Author

@hohwille I have applied the changes. The code is now more compact. Thank you for the comments.

Copy link
Member

@hohwille hohwille left a 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.

@hohwille hohwille merged commit ffd6bad into devonfw:master May 31, 2022
@hohwille hohwille added this to the release:2022.04.002 milestone May 31, 2022
@alfeilex alfeilex deleted the feature/azurecli branch June 1, 2022 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bash related to bash shell or scripts commandlet related to commandlets (scripts/command/*) documentation related to documentation (AsciiDoc) scripts related to shell scripts (bash and CMD)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Azure CLI integration
4 participants