Skip to content

Make JSON machine readable#144

Merged
marciniwanicki merged 2 commits into
macvmio:mainfrom
beefon:dev/beefon/JsonMachineReadableFormatting
Jan 1, 2026
Merged

Make JSON machine readable#144
marciniwanicki merged 2 commits into
macvmio:mainfrom
beefon:dev/beefon/JsonMachineReadableFormatting

Conversation

@beefon
Copy link
Copy Markdown
Contributor

@beefon beefon commented Dec 31, 2025

Text output is intact:

$ ./.build/debug/curie images 

REPOSITORY                    TAG     IMAGE ID         CREATED          SIZE     
myteam/myimage/test-image     2.0     c959e4645f12     3 months ago     120.03 GB
myteam/myimage/test-image     1.0     c0db27f611d0     3 months ago     120.03 GB

JSON output has been updated:

$ ./.build/debug/curie images -f json
[
  {
    "created" : "2025-09-28T20:57:10Z",        <<<< this is machine readable now
    "image_id" : "c959e4645f12",
    "repository" : "myteam\/myimage\/test-image",
    "size" : {                                 <<<< this is machine readable now
      "bytes" : 128882598851
    },
    "tag" : "2.0"
  },
  {
    "created" : "2025-09-28T20:52:34Z",
    "image_id" : "c0db27f611d0",
    "repository" : "myteam\/myimage\/test-image",
    "size" : {
      "bytes" : 128882598832
    },
    "tag" : "1.0"
  }
]

Test Plan:

  • Ensure all CI checks pass
  • Run both curie images and curie images -f json, verify the text version of the command prints human-friendly timestamp (e.g. 2 days ago) and JSON prints ISO8601 format

@marciniwanicki
Copy link
Copy Markdown
Contributor

The change looks great, thanks @beefon! Let's fix the lint issue and if we could remove the emoji from the description and perhaps add one line about what changed and the difference tween human readable table vs json that would be great.

@marciniwanicki
Copy link
Copy Markdown
Contributor

I've just noticed we have the same lint issue on main, will check it.

@marciniwanicki marciniwanicki force-pushed the dev/beefon/JsonMachineReadableFormatting branch from 5caae95 to 15ac46f Compare January 1, 2026 19:38
@marciniwanicki marciniwanicki merged commit af0d6d1 into macvmio:main Jan 1, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants