docs(changelog): backfill omitted 0.8.1 and 0.9.0 entries - #104
Conversation
|
|
||
| ### Changed | ||
|
|
||
| - **Breaking:** `LoadManagedTableRequest` can now load from a persisted query |
There was a problem hiding this comment.
super nit: (not blocking) The published GitHub Release notes for v0.8.1 and v0.9.0 won't pick these up. .github/workflows/release.yml checks out ref: ${{ env.RELEASE_TAG }} (line 37) and extracts notes from the CHANGELOG at that tag, so a workflow_dispatch re-run for v0.8.1 would re-render the old, pre-backfill section — and make_latest: true (line 78) would additionally re-point "latest" at an old tag. Since the point of the backfill is that affected callers were never told, it's worth editing those two release bodies by hand after merge.
There was a problem hiding this comment.
Verified every claim against the current source: DatabaseSummary::new(default_catalog, default_schema, id), DatabaseDetailResponse::new(attachments, default_catalog, default_connection_id, default_schema, id), CreateDatabaseResponse::new(default_catalog, default_connection_id, default_schema, id), the optional default_schema on CreateDatabaseRequest, the optional key on AddManagedTableDecl / AddManagedTableRequest / DatabaseDefaultTableDecl, and LoadManagedTableRequest::new(mode) with optional upload_id / result_id. The 0.9.0 attribution matches PR #90's file list (merged 2026-07-15, same day as the tag). Semantics in the new entries match the generated doc comments.
|
Confirmed, and worth sharpening: the hazard isn't only stale notes. Both published bodies currently carry only the raw subjects, so they need the same treatment as the CHANGELOG sections here. Doing that after this merges. Separately, the unconditional |
Two shipped releases carried breaking changes recorded only as raw commit subjects, so the changelog never told callers they were affected. Backfills the missing entries; no code changes.
0.9.0 — regen #90 added a required
default_schemafield toDatabaseSummary,DatabaseDetailResponse, andCreateDatabaseResponse, which changed all threenew()signatures. It also added optionaldefault_schemaonCreateDatabaseRequestand optionalkeyon three table decls. The only entry waschore(api): exclude datasets from public OpenAPI spec + docs cleanup.0.8.1 —
LoadManagedTableRequest::new(mode, upload_id)becamenew(mode). This went out in a patch release, which Cargo treats as compatible with 0.8.0 for a0.xcrate, so it could surface oncargo update. The entry wasfeat(tables): support loading from query results.Every claim was verified against the
v0.8.1/v0.9.0tags rather than the PR diffs. Existing raw-subject bullets are left in place so the entries stay traceable to their regens.