-
Notifications
You must be signed in to change notification settings - Fork 221
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
Json output for the kiota info
command
#3531
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.
Yes that works better actually! Thanks for making the change.
@andreaTP would you mind adding the documentation for the new option in the documentation as well please? |
@@ -101,6 +101,7 @@ private static Command GetInfoCommand() | |||
var clearCacheOption = GetClearCacheOption(defaultGenerationConfiguration.ClearCache); | |||
var searchTermOption = GetSearchKeyOption(); | |||
var languageOption = new Option<GenerationLanguage?>("--language", "The target language for the dependencies instructions."); | |||
var jsonOption = new Option<bool>("--json", "The target language for the dependencies instructions."); |
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.
Ouch, I just noticed this, I need a better description for this.
This should ease the integrations and enable them to better automate processes.