Skip to content

Enable customization of table borders in ~/.myclirc#1664

Merged
rolandwalker merged 1 commit intomainfrom
RW/set-up-table-border-color-config
Mar 1, 2026
Merged

Enable customization of table borders in ~/.myclirc#1664
rolandwalker merged 1 commit intomainfrom
RW/set-up-table-border-color-config

Conversation

@rolandwalker
Copy link
Contributor

Description

Styling table borders with a Pygments style turns out to have been already implemented in cli_helpers, per

In the mycli project, we just need to define Token.Output.TableSeparator and make it configurable by the user in ~/.myclirc.

xref #1663, #1654, #1653

Example with dimmed table borders:

Screenshot 2026-02-28 at 3 05 25 PM

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@github-actions
Copy link

  1. Low: Missing regression test for the new output.table-separator mapping

No correctness or security issues were evident in this PR diff beyond the test gap above.

@rolandwalker rolandwalker self-assigned this Feb 28, 2026
Styling table borders with a Pygments style turns out to have been
already implemented in cli_helpers, per

 * https://github.com/dbcli/cli_helpers/blob/9fb9f656ea8f4ab8230c2e4633526791d2d7438e/cli_helpers/tabular_output/tabulate_adapter.py#L163

In the mycli project, we just need to define Token.Output.TableSeparator
and make it configurable by the user in ~/.myclirc.
@rolandwalker rolandwalker force-pushed the RW/set-up-table-border-color-config branch from d5e9499 to 84af474 Compare February 28, 2026 23:57
@rolandwalker rolandwalker merged commit a76b9a3 into main Mar 1, 2026
8 checks passed
@rolandwalker rolandwalker deleted the RW/set-up-table-border-color-config branch March 1, 2026 00:02
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