Skip to content

Display byte units in text list/get output - #1265

Closed
nanookclaw wants to merge 1 commit into
openai:mainfrom
nanookclaw:fix/list-human-byte-units
Closed

Display byte units in text list/get output#1265
nanookclaw wants to merge 1 commit into
openai:mainfrom
nanookclaw:fix/list-human-byte-units

Conversation

@nanookclaw

Copy link
Copy Markdown

Updates the text render paths for tart list and tart get so their Disk and Size values are shown with ByteCountFormatter human-readable units, while preserving the existing field and column names.

The JSON paths keep their previous machine-readable behavior for compatibility: tart list --format json still emits integer GB values for Disk and Size, and tart get --format json still emits integer GB for Disk plus the existing three-decimal GB string for Size.

Focused tests cover both compatibility shapes and the human-readable text projections.

Fixes #820

Testing

  • git diff --check
  • Not run: swift test --filter 'GetTests|ListTests' (swift is not installed in this environment)

@edi-oai edi-oai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, perhaps I'm missing the point, but it feels like we can do this much simpler and minimize the overall churn.

What do you think about #1301 as an alternative implementation?

@edi-oai edi-oai closed this in #1301 Aug 5, 2026
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.

Suggestion: display disk and size attributes with appropriate byte size units

3 participants