Skip to content

Show truncation indicator in array display#7072

Open
dimitarvdimitrov wants to merge 1 commit intodevelopfrom
mitko/display/truncation-indicator
Open

Show truncation indicator in array display#7072
dimitarvdimitrov wants to merge 1 commit intodevelopfrom
mitko/display/truncation-indicator

Conversation

@dimitarvdimitrov
Copy link
Contributor

When displaying array values that exceed the display limit (16), we now show ... at the end instead of silently truncating. Makes it obvious the output is incomplete.

@dimitarvdimitrov dimitarvdimitrov added changelog/skip Do not list PR in the changelog changelog/fix A bug fix and removed changelog/fix A bug fix labels Mar 19, 2026
@dimitarvdimitrov dimitarvdimitrov force-pushed the mitko/display/truncation-indicator branch from 6378a6d to 1c79cc4 Compare March 19, 2026 18:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates array value display so that when the displayed values are truncated (default limit 16), the output visibly indicates truncation by appending an ellipsis.

Changes:

  • Update scalar array display formatting to append ... when output is truncated.
  • Update DisplayOptions docs to mention the default truncation behavior.
  • Add a unit test asserting the new truncation indicator behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Dimitar Dimitrov <dimitar@spiraldb.com>
@dimitarvdimitrov dimitarvdimitrov force-pushed the mitko/display/truncation-indicator branch from 1c79cc4 to 0e2ed56 Compare March 19, 2026 18:13
@dimitarvdimitrov dimitarvdimitrov marked this pull request as ready for review March 19, 2026 18:21
Copy link
Contributor

@joseph-isaacs joseph-isaacs left a comment

Choose a reason for hiding this comment

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

we need to add custom trunc len. but this can be a follow up

Copy link
Contributor

@joseph-isaacs joseph-isaacs left a comment

Choose a reason for hiding this comment

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

would be nice to include the last few values too?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/skip Do not list PR in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants