Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SQL Migration query builder API and migration API #108

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

christianparpart
Copy link
Member

@christianparpart christianparpart commented Nov 25, 2024

Refs #6

checklist (query builder)

each item should have its own test

  • CreateTable()
  • DropTable()
  • AlterTable()
  • AlterTable: RenameTable
  • AlterTable: RenameColumn
  • AlterTable: AddColumn
  • AlterTable: DropColumn
  • AlterTable: AddIndex
  • AlterTable: DropIndex
  • make use of Migration query builder in existing tests to avoid writing raw CREATE TABLE statements.

based on that, we can build a DB migration API that makes use of the fluent API to express schema changes.

@christianparpart christianparpart added CLI command line interface tools Query Builder labels Nov 25, 2024
@christianparpart christianparpart force-pushed the feature/migrations branch 10 times, most recently from d40ecfe to 0af1dc4 Compare December 2, 2024 20:56
@christianparpart christianparpart marked this pull request as ready for review December 2, 2024 20:57
@christianparpart christianparpart force-pushed the feature/migrations branch 3 times, most recently from 46ba176 to 3d61d11 Compare December 2, 2024 21:33
… use of it in unit tests

Signed-off-by: Christian Parpart <[email protected]>
@christianparpart christianparpart merged commit 7322c9b into master Dec 2, 2024
9 checks passed
@christianparpart christianparpart deleted the feature/migrations branch December 2, 2024 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI command line interface tools Query Builder
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant