Skip to content

Add DeleteDescendants option to branch delete requests#297

Merged
nholden merged 1 commit intomainfrom
delete-descendants
Mar 27, 2026
Merged

Add DeleteDescendants option to branch delete requests#297
nholden merged 1 commit intomainfrom
delete-descendants

Conversation

@nholden
Copy link
Copy Markdown
Member

@nholden nholden commented Mar 27, 2026 •
edited
Loading

Summary

  • Adds a DeleteDescendants bool field to DeleteDatabaseBranchRequest and DeletePostgresBranchRequest
  • When set, the Delete methods pass ?delete_descendants=true as a query parameter to the API
  • Backward compatible — the zero-value false preserves existing behavior

Test plan

  • Unit tests for both MySQL and PostgreSQL delete (with and without descendants)
  • Manual test via CLI against a branch with descendants

🤖 Generated with Claude Code

@nholden @claude
Add DeleteDescendants option to branch delete requests
9217eb5 
When set, passes a `delete_descendants=true` query parameter to the
API DELETE endpoint, allowing deletion of a branch and all of its
descendant branches in a single operation.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nholden nholden mentioned this pull request Mar 27, 2026
Merged
3 tasks
@nholden nholden self-assigned this Mar 27, 2026
@nholden nholden marked this pull request as ready for review March 27, 2026 21:53
@nholden nholden requested a review from a team as a code owner March 27, 2026 21:53
@nholden nholden merged commit 8f52eb6 into main Mar 27, 2026
5 checks passed
@nholden nholden deleted the delete-descendants branch March 27, 2026 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mscoutermarsh mscoutermarsh mscoutermarsh approved these changes

Assignees

@nholden nholden

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nholden @mscoutermarsh