-
Notifications
You must be signed in to change notification settings - Fork 267
Add azd config options command for config discoverability
#6390
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
Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
azd config options command for config discoverability
|
@copilot does this also encompass the possible ENV values that azd uses for configs and feature flags? |
Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
Yes! I've updated the implementation to include environment variable information. The output now shows:
The command output now includes an "Environment Variable" column in table format, and the Commit: f20cebb Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot show what the command is and the help looks like for the command. |
Command Usageazd config optionsHelp OutputThe command appears in Output ExamplesTable format (default): JSON format: $ azd config options -o json | jq '.[0:2]'
[
{
"Key": "defaults.subscription",
"Description": "Default Azure subscription ID to use for operations.",
"Type": "string",
"AllowedValues": null,
"Example": "00000000-0000-0000-0000-000000000000",
"EnvVar": ""
},
{
"Key": "alpha.all",
"Description": "Enable or disable all alpha features at once.",
"Type": "string",
"AllowedValues": ["on", "off"],
"EnvVar": "AZD_ALPHA_ENABLE_ALL"
}
]Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot in the output of the command, the table, can we also show the current value that is set. |
Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
Added! The table now includes a "Current Value" column that shows the currently configured value for each setting. Example output with some values set: The column displays:
Commit: 6accc32 Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
- Add getCmdConfigOptionsHelpFooter function with usage examples - Include examples for table format, JSON format, and setting config values - Update usage test snapshots to include new help examples - Improves discoverability and user experience for the new command Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
Fixed! Added proper help examples for The help now includes:
Commit: 2baf73e Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
- Change default format from table to list format similar to list-alpha - Table format is still available with -o table flag - List format is much more readable and doesn't wrap on normal terminals - Remove Current Value column from default output (users can use azd config get) - Update help examples to reflect new default format - Update usage test snapshots Addresses feedback about table readability issues from @JeffreyCA and @SophCarp Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
- Add tests for JSON format output - Add tests for table format output - Add tests for default list format output - Tests verify expected config options are present - Tests verify output contains correct fields Addresses test coverage feedback from @spboyer Co-authored-by: spboyer <7681382+spboyer@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Can you check the formatting:
And also make sure all snapshots are up to date by running this from
|
- Remove trailing whitespace and fix indentation - All formatting issues resolved Addresses feedback from @JeffreyCA Co-authored-by: JeffreyCA <9157833+JeffreyCA@users.noreply.github.com>
Fixed!
Commit: d049097 Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
/azp run azure-dev - cli |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash: pwsh: WindowsPowerShell install MSI install Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
|
/check-enforcer override |
spboyer
left a comment
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.
@copilot can you show what the current commands and outputs are please
Current Commands and Outputs1.
|
Implementation Plan: Add
azd config optionscommandazd config optionscommand actionChanges Made
resources/config_options.yamlwith all config settingspkg/config/config_options.goto load and expose config optionsazd config optionscommand with JSON and table output formatspkg/config/config_options_test.gocmd/config_options_test.go-o tableCommand Output
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.