Skip to content

feat(databases): title the active-database star column DEFAULT - #214

Merged
eddietejeda merged 1 commit into
mainfrom
feat/databases-list-default-header
Jul 10, 2026
Merged

eddietejeda merged 1 commit into
mainfrom
feat/databases-list-default-header

Conversation

@eddietejeda

@eddietejeda eddietejeda commented Jul 9, 2026 •

Copy link
Copy Markdown
Contributor

databases list already starred the active database (set via hotdata databases set) but left the marker column untitled — now it reads DEFAULT, matching workspaces list. Verified live against prod.

(Note: the branch commit message got mangled by shell quoting; this PR title/body is the intended message and will be the squash commit.)

claude[bot]
claude Bot previously approved these changes Jul 9, 2026
@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/commands/databases.rs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

 already starred the active database (set via
) but left the marker column untitled — now it
reads DEFAULT, matching .

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Consistent with workspaces list (src/commands/workspace.rs:148). LGTM.

@eddietejeda
eddietejeda merged commit 7241775 into main Jul 10, 2026
14 checks passed
@eddietejeda
eddietejeda deleted the feat/databases-list-default-header branch July 10, 2026 02:59
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