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

Initial database schema #100

Merged
merged 1 commit into from
Jul 26, 2020
Merged

Initial database schema #100

merged 1 commit into from
Jul 26, 2020

Conversation

malor
Copy link
Member

@malor malor commented Jul 25, 2020

Diesel is used for schema migrations (written in raw SQL; targetting
PostgreSQL at the moment) and conversions between SQL and Rust
data types.

@malor malor requested a review from ikalnytskyi July 25, 2020 16:24
@malor malor force-pushed the diesel branch 3 times, most recently from 29609be to 6b595b9 Compare July 25, 2020 17:05
@malor malor force-pushed the diesel branch 3 times, most recently from 86a70e8 to 82133d3 Compare July 25, 2020 21:28
Diesel is used for schema migrations (written in raw SQL; targetting
PostgreSQL at the moment) and conversions between SQL and Rust
datatypes.
@@ -16,5 +16,7 @@ categories = [
publish = false

[dependencies]
chrono = "0.4.13"
diesel = { version = "1.4.5", features = ["postgres"] }
Copy link
Member

Choose a reason for hiding this comment

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

Q: Do we need to install diesiel-cli from master?

Copy link
Member Author

Choose a reason for hiding this comment

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

My current understanding is that this is not possible at the moment, because diesel_cli is a binary-only crate :( rust-lang/cargo#4316

Let me try using https://docs.rs/diesel_migrations/1.4.0/diesel_migrations/index.html in a follow up PR. Or just replace this with alembic ;)

@malor malor merged commit 49b7cba into master Jul 26, 2020
@malor malor deleted the diesel branch July 26, 2020 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants