Skip to content

chore(deps): bump hotdata SDK to 0.19.0 - #317

Merged
shefeek-jinnah merged 1 commit into
mainfrom
chore/sdk-0.19
Sep 23, 2026
Merged

shefeek-jinnah merged 1 commit into
mainfrom
chore/sdk-0.19

Conversation

@shefeek-jinnah

Copy link
Copy Markdown
Contributor

Summary

Bumps the hotdata SDK from 0.18.1 to 0.19.0. This PR changes no CLI behavior.

  • Test fixtures: 0.19.0 makes constant_per_key a required field on TableInfo. The server has sent it since runtimedb #1300 (Sept 3), but the CLI's hand-written table-listing fixtures didn't, so they are updated to match.
  • ForkDatabaseRequest: it has a new description field. The CLI leaves it unset (None) here. feat(databases): record why a fork was taken #315 adds the --description flag that sets it.

#315 builds on this PR.

Test plan

  • cargo test: 596 passed
  • cargo clippy --all-targets -- -D warnings and cargo fmt --check

0.19.0 makes constant_per_key a required field on TableInfo. The server
already sends it, so this updates the hand-written table-listing test
fixtures to match. ForkDatabaseRequest gains a description field, which
the CLI leaves unset for now.
@shefeek-jinnah
shefeek-jinnah requested a review from a team as a code owner September 23, 2026 11:43
@shefeek-jinnah
shefeek-jinnah requested review from zfarrell and removed request for a team September 23, 2026 11:43
Comment thread src/commands/databases.rs
hotdata::models::ForkDatabaseRequest {
name: name.map(Some),
expires_at: expires_at.map(|e| Some(e.to_string())),
description: None,

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.

super nit: the comment above at line 882 says "both fields", but the struct literal now sets three fields. Update the wording to cover description so the double-option note stays accurate. (not blocking)

@codecov

codecov Bot commented Sep 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@shefeek-jinnah
shefeek-jinnah merged commit 70033de into main Sep 23, 2026
15 checks passed
@shefeek-jinnah
shefeek-jinnah deleted the chore/sdk-0.19 branch September 23, 2026 11:52
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