Skip to content

Probe tabulate version for preserve_whitespace#109

Merged
rolandwalker merged 1 commit intomainfrom
RW/probe-tabulate-preserve-whitespace
Mar 20, 2026
Merged

Probe tabulate version for preserve_whitespace#109
rolandwalker merged 1 commit intomainfrom
RW/probe-tabulate-preserve-whitespace

Conversation

@rolandwalker
Copy link
Contributor

Description

Somehow, users are able to have installations of newer mycli/cli_helpers with older versions of tabulate, before a breaking change in the way preserve_whitespace is specified for tabulate.

Here we probe the tabulate version to determine how to specify the value.

Checklist

  • I've added this contribution to the CHANGELOG.
  • I've added my name to the AUTHORS file (or it's already there).
  • I installed pre-commit hooks (pip install pre-commit && pre-commit install), and ran black on my code.
  • Please squash merge this pull request (uncheck if you'd like us to merge as multiple commits)

Somehow, users are able to have installations of newer mycli/cli_helpers
with older versions of tabulate, before a breaking change in the way
preserve_whitespace is specified for tabulate.

Here we probe the tabulate version to determine how to specify the
value.
@rolandwalker rolandwalker self-assigned this Mar 20, 2026
@rolandwalker
Copy link
Contributor Author

Should fix dbcli/mycli#1699 .

@rolandwalker rolandwalker merged commit 4d79cc1 into main Mar 20, 2026
2 checks passed
@rolandwalker rolandwalker deleted the RW/probe-tabulate-preserve-whitespace branch March 20, 2026 21:55
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