Skip to content

Update database schema to match the code #1133

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

Merged
merged 1 commit into from
Oct 24, 2017

Conversation

sgrif
Copy link
Contributor

@sgrif sgrif commented Oct 16, 2017

There are a handful of differences between what we say our schema is in
the code, and what it actually is. These differences make schema.rs
harder to regenerate, and can potentially lead to real issues down the
line. This removes all of the differences between the real schema and
what we say it is in Rust.

There are a handful of differences between what we say our schema is in
the code, and what it actually is. These differences make schema.rs
harder to regenerate, and can potentially lead to real issues down the
line. This removes all of the differences between the real schema and
what we say it is in Rust.
@carols10cents
Copy link
Member

So this is better, but when I'm on this branch and run diesel setup and diesel print-schema --with-docs > src/schema.rs, the use lines added here are just removed again :-/ So this seems better as far as ease of regeneration, but not quite ideal...

@sgrif
Copy link
Contributor Author

sgrif commented Oct 22, 2017

Yeah, there's not really any way for diesel print-schema to know exactly what we've changed. As I mentioned on another PR, I think it might be worth us keeping a .patch file around for any changes that we do make to this file to make it easier to regenerate, but I don't think that should block this PR.

@carols10cents
Copy link
Member

Ok, I'll think about how to handle the other issues and merge this in the meantime.

bors: r+

bors-voyager bot added a commit that referenced this pull request Oct 24, 2017
1133: Update database schema to match the code r=carols10cents

There are a handful of differences between what we say our schema is in
the code, and what it actually is. These differences make schema.rs
harder to regenerate, and can potentially lead to real issues down the
line. This removes all of the differences between the real schema and
what we say it is in Rust.
@bors-voyager
Copy link
Contributor

bors-voyager bot commented Oct 24, 2017

Build succeeded

@bors-voyager bors-voyager bot merged commit 3f9416e into rust-lang:master Oct 24, 2017
@sgrif sgrif deleted the sg-make-schema-match-code branch December 4, 2017 20:35
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