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

Working drizzle integration #34

Closed
wants to merge 1 commit into from
Closed

Conversation

Hajbo
Copy link
Collaborator

@Hajbo Hajbo commented Sep 24, 2023

A quick PoC for the drizzle ORM. Unfortunately drizzle-kit doesn't support bun:sqlite so for the migration generation we need to use better-sqlite for now.

To create a migration:

  1. Make sure you are in the root of the project
  2. Run bun drizzle-kit generate:sqlite

To run the example and auto-apply the migration:

cd src/db
bun index.ts

It'll create the users table (if needed), insert a new user and select all existing users.

@Hajbo Hajbo requested review from yamcodes and aruaycodes and removed request for yamcodes September 24, 2023 11:43
@yamcodes
Copy link
Contributor

yamcodes commented Sep 25, 2023

This was internally scrapped in favor of #35, due to Drizzle's poor support of file/in-memory databases in Bun. (At least Sqlite based ones.)

We should create a minimal reproducible example bug report for the Drizzle team.

@yamcodes yamcodes closed this Sep 25, 2023
@yamcodes yamcodes deleted the feature/bun-drizzle-sqlite branch September 25, 2023 16:29
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