Skip to content

fix: restore output for add and remove subcommands, add tests#348

Merged
zeroshade merged 1 commit intocolumnar-tech:mainfrom
amoeba:fix/dbc-add-not-rendering
Apr 7, 2026
Merged

fix: restore output for add and remove subcommands, add tests#348
zeroshade merged 1 commit intocolumnar-tech:mainfrom
amoeba:fix/dbc-add-not-rendering

Conversation

@amoeba
Copy link
Copy Markdown
Member

@amoeba amoeba commented Apr 7, 2026

In our upgrade to bubbletea v2, we didn't catch that we lost the output for dbc add and remove. Whereas dbc 0.2.0 prints like this,

$ dbc add mysql
added mysql to driver list
use `dbc sync` to install the drivers in the list

git main prints like this (nothing):

$ dbc add mysql

This patch restores the output and adds regression tests for both subcommands.

@amoeba amoeba requested a review from zeroshade April 7, 2026 00:00
@amoeba amoeba force-pushed the fix/dbc-add-not-rendering branch from fadb0ba to 35bfd11 Compare April 7, 2026 00:43
Copy link
Copy Markdown
Member

@zeroshade zeroshade left a comment

Choose a reason for hiding this comment

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

Nice catch!

@zeroshade zeroshade merged commit 9bb8187 into columnar-tech:main Apr 7, 2026
11 checks passed
zeroshade pushed a commit that referenced this pull request Apr 17, 2026
In our upgrade to bubbletea v2, we didn't catch that we lost the output
for dbc add and remove. Whereas dbc 0.2.0 prints like this,

```sh
$ dbc add mysql
added mysql to driver list
use `dbc sync` to install the drivers in the list
```

git main prints like this (nothing):

```sh
$ dbc add mysql
```

This patch restores the output and adds regression tests for both
subcommands.
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