Skip to content

feat: drop objects#18

Merged
leiysky merged 2 commits intofast:mainfrom
leiysky:feat/drop-index
Feb 2, 2026
Merged

feat: drop objects#18
leiysky merged 2 commits intofast:mainfrom
leiysky:feat/drop-index

Conversation

@leiysky
Copy link
Copy Markdown
Contributor

@leiysky leiysky commented Feb 2, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds SQL builders for DROP INDEX, DROP TABLE, and DROP SCHEMA, including support for IF EXISTS and drop behavior (CASCADE / RESTRICT), plus snapshot tests to validate generated SQL.

Changes:

  • Introduce DropBehavior enum to represent CASCADE / RESTRICT.
  • Add new statement builders: DropIndex, DropTable, DropSchema.
  • Add insta snapshot tests covering the generated DROP SQL.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pqb/tests/drop.rs Adds snapshot tests validating DROP SQL generation.
pqb/src/types/mod.rs Introduces DropBehavior shared by DROP statement builders.
pqb/src/table/mod.rs Wires in the new table::drop module and re-exports it.
pqb/src/table/drop.rs Implements DropTable builder and SQL writer.
pqb/src/schema/mod.rs Introduces new schema module with drop support.
pqb/src/schema/drop.rs Implements DropSchema builder and SQL writer.
pqb/src/lib.rs Exposes the new schema module from the crate root.
pqb/src/index/mod.rs Wires in the new index::drop module and re-exports it.
pqb/src/index/drop.rs Implements DropIndex builder and SQL writer.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@leiysky leiysky merged commit ee79972 into fast:main Feb 2, 2026
9 checks passed
@leiysky leiysky deleted the feat/drop-index branch February 2, 2026 15:12
@leiysky leiysky review requested due to automatic review settings March 23, 2026 22:16
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