Add container system version with format yaml#1353
Add container system version with format yaml#1353tico88612 wants to merge 2 commits intoapple:mainfrom
Conversation
|
@tico88612 Adding YAML as a format option is a nice addition, but could you have a look at all the different |
Maybe I will send another PR to do this. |
jglogan
left a comment
There was a problem hiding this comment.
No changes are necessary on this one but we need to wait on merging the Package.swift change that includes Yams. Should be a matter of days.
|
I was gonna consolidate the formatting logic into one - was thinking we'd extract the if json / if quiet / else table rendering logic from each list command into a |
|
Related #1385, change to WIP until the PR is ready. |
|
@tico88612 #1385 is merged now |
|
@realrajaryan Thanks, I will update later. |
3dc5713 to
e5b9fe8
Compare
|
@realrajaryan Done. You can review this again. |
|
@tico88612 Thanks for the update! Could you check this test fail? |
|
@tico88612 probably should |
Signed-off-by: ChengHao Yang <17496418+tico88612@users.noreply.github.com>
Signed-off-by: ChengHao Yang <17496418+tico88612@users.noreply.github.com>
e5b9fe8 to
bafab46
Compare
Type of Change
Motivation and Context
Fixed #1156
Testing