Repository navigation
Change alias list output format to YAML - #6603
Conversation
cliAutomation
added
the
external
mislav
left a comment
mislav
left a comment
There was a problem hiding this comment.
Thanks for taking this on! Let's use YAML as it's more readable in its raw format than JSON.
|
@mislav Sure thing! I should have enough time to do that in the next day or two. |
|
@mislav I changed the output format to yaml and am looking into why the value of the gc alias is getting single-quoted (it's not occurring outside the test in command prompt, powershell, or bash). |
|
@mislav Quick question here, do we want to keep the table output for TTY mode and just output YAML when in NOTTY mode? |
There was a problem hiding this comment.
This looks good to me; thank you! I've pushed a change that preserves newlines in alias expansions, since if we scrub them we risk breaking the expansion script.
@samcoe For simplicity sake, let's just output the same YAML in both non-TTY and TTY modes. Table output doesn't play well with multiline aliases anyway, and YAML handles it reasonably well.
mislav
changed the title
4 participants
Fixes #6497
I used JSON instead of YAML as the output format, but if there's a preference towards YAML I can update it.