-
-
Notifications
You must be signed in to change notification settings - Fork 808
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
Diesel-async usage example #992
Conversation
@robjtede could you help resolve the problem of installing the required |
Yep I'll check this out later. |
please feel free to squash and/or perform a rebase/force push on your branch please try not to update unrelated workspace deps in the future, the conflicts have become very complicated to resolve cleanly as a result |
….md to check if postgres is up and running
eda6e97
to
8681b2a
Compare
Done |
@robjtede What will we do with the tests? Will you mark them as ignored or add the small (alpine?) PostgreSQL instance setup to the CI/CD pipeline? |
Maybe we could mark as ignored? Alternatively rewrite this to use sqlite which is how we've gotten around needing DBs running in CI with the other examples. |
Diesel-async support only PostgreSQL and MySQL. |
6343c45
to
9a25c3b
Compare
@robjtede okay, it's done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good addition, thanks 👍🏻
i may make some tweaks once merged so as not to bore you with nits
I think it's useful nowadays to have a code example like this as a starting point.