-
Notifications
You must be signed in to change notification settings - Fork 228
Add azd env get-value #4080
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
Add azd env get-value #4080
Conversation
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.
Cool, thank you!
@pamelafox - Could you please install the build from this PR and see if it works for you? |
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.
Looks great, just added a couple points that we should agree on before finishing up.
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash:
pwsh:
WindowsPowerShell install
MSI install
Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
I just tried it out, worked great! Much nicer than my grepsed horror. |
@pamelafox , can you share the way you are using I'm curious to understand what's the use case and if the |
@vhvb1989 From a script that's used in hooks (or outside of hooks)- |
It would also be useful in CI, like this workflow from @tonybaloney - |
@pamelafox , thanks for sharing. So, the script is typically expected to be called by azd from a hook, but you want to allow folks to run the script w/o calling either Are you not using What if From my POV, I would much prefer if the scripts pull the value directly from the system environment and having azd to inject that env during the script execution. That's for the benefit of using the script w/o azd as well. |
That's an interesting approach! |
@vhvb1989 Your summary is correct, I want to be able to run the scripts separately, versus all three at once that the hook does. I think it'd be nice for |
* Test aks * Add get-value * Undo aks string * Remove formatter * Add tests
Adds
azd env get-value keyName
command.Fixes #3906