Skip to content

✨ feat(cli): auto-expand table dicts to markdown nodes in CLI output#1490

Merged
harehare merged 1 commit intomainfrom
feat/table-auto-expand
Mar 24, 2026
Merged

✨ feat(cli): auto-expand table dicts to markdown nodes in CLI output#1490
harehare merged 1 commit intomainfrom
feat/table-auto-expand

Conversation

@harehare
Copy link
Copy Markdown
Owner

Table objects returned from table::tables() are now automatically expanded to markdown nodes when output via the CLI, matching the existing behavior for section dicts. Users can output table objects directly without calling table::to_markdown().

Also fixes a bug in table::add_row() where new cells were assigned the same row index as existing data rows due to missing +1 offset for the header row.

Base automatically changed from fix/typechecker-coalesce-range-operators to main March 24, 2026 06:30
Table objects returned from table::tables() are now automatically
expanded to markdown nodes when output via the CLI, matching the
existing behavior for section dicts. Users can output table objects
directly without calling table::to_markdown().

Also fixes a bug in table::add_row() where new cells were assigned
the same row index as existing data rows due to missing +1 offset
for the header row.
@harehare harehare force-pushed the feat/table-auto-expand branch from 4ce1f35 to 3649a2f Compare March 24, 2026 06:30
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Mar 24, 2026

Merging this PR will not alter performance

✅ 29 untouched benchmarks


Comparing feat/table-auto-expand (3649a2f) with main (edc228e)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (f978e5b) during the generation of this report, so edc228e was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@harehare harehare merged commit 9253c18 into main Mar 24, 2026
6 checks passed
@harehare harehare deleted the feat/table-auto-expand branch March 24, 2026 07:21
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.

1 participant