-
Notifications
You must be signed in to change notification settings - Fork 4
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
Dependency on sqlx #1
Comments
I just find the code easier to read and write. |
It's just one more dependency a user has to pull in. Not everybody is using sqlx in their project, so they would have that dependency only for the migration part. Really the only thing that would change is the select in |
Understood. Ok, go for it then. |
Will do on the weekend |
Apart from legacy, is there a reason we have to depend on sqlx? If not, I would tackle removing it on the weekend.
The text was updated successfully, but these errors were encountered: