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

Fix "Trait object without an explicit dyn is deprecated" warnings #223

Merged
merged 2 commits into from
Jul 20, 2019
Merged

Fix "Trait object without an explicit dyn is deprecated" warnings #223

merged 2 commits into from
Jul 20, 2019

Conversation

Bunogi
Copy link
Contributor

@Bunogi Bunogi commented Jul 20, 2019

A recent-ish version of Rust nightly deprecated bare trait objects. This PR adds dyn to all trait objects in order to get rid of the warning.

@badboy
Copy link
Collaborator

badboy commented Jul 20, 2019

This bumps our minimum required Rust version to 1.27.
I'm fine with that. Would you be so kind changing the note in the README in this PR?

@Bunogi
Copy link
Contributor Author

Bunogi commented Jul 20, 2019

@badboy No problem at all, done.

@badboy badboy merged commit c615ec4 into redis-rs:master Jul 20, 2019
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