Skip to content

Update to Diesel 1.1 #1234

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 2 commits into from
Jan 18, 2018
Merged

Update to Diesel 1.1 #1234

merged 2 commits into from
Jan 18, 2018

Conversation

sgrif
Copy link
Contributor

@sgrif sgrif commented Jan 15, 2018

This is the minimum changes required to fix any deprecation warnings.
There's a lot of additional stuff we could do that I wanted to keep
separate. After updating to Diesel 1.1, we can remove virtually anywhere
that we're doing an explicit &*. Additionally, there are probably a
lot of places that we should be using custom types but aren't. We can
remove a lot of code once
dtolnay/semver#169 is merged/released as well.

sgrif and others added 2 commits January 15, 2018 09:29
This is the minimum changes required to fix any deprecation warnings.
There's a lot of additional stuff we could do that I wanted to keep
separate. After updating to Diesel 1.1, we can remove virtually anywhere
that we're doing an explicit `&*`. Additionally, there are probably a
lot of places that we should be using custom types but aren't. We can
remove a lot of code once
dtolnay/semver#169 is merged/released as well.
@carols10cents
Copy link
Member

\o/

bors: r+

bors-voyager bot added a commit that referenced this pull request Jan 18, 2018
1234: Update to Diesel 1.1 r=carols10cents

This is the minimum changes required to fix any deprecation warnings.
There's a lot of additional stuff we could do that I wanted to keep
separate. After updating to Diesel 1.1, we can remove virtually anywhere
that we're doing an explicit `&*`. Additionally, there are probably a
lot of places that we should be using custom types but aren't. We can
remove a lot of code once
dtolnay/semver#169 is merged/released as well.
@bors-voyager
Copy link
Contributor

bors-voyager bot commented Jan 18, 2018

Build succeeded

@bors-voyager bors-voyager bot merged commit afff732 into rust-lang:master Jan 18, 2018
@sgrif sgrif deleted the sg-diesel-1.1 branch January 18, 2018 16:39
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