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

Update to Rust 2018 #445

Merged
merged 5 commits into from
Dec 7, 2018
Merged

Conversation

Lakelezz
Copy link
Contributor

@Lakelezz Lakelezz commented Dec 7, 2018

This pull request updates Serenity's code to utilise Rust 2018, including examples.
Additionally updates sections where & on match and if let is not needed due to automatic dereferencing.

@Lakelezz Lakelezz added the enhancement An improvement to Serenity. label Dec 7, 2018
@Lakelezz Lakelezz added this to the 0.6 milestone Dec 7, 2018
@Lakelezz Lakelezz mentioned this pull request Dec 7, 2018
18 tasks
@arqunis
Copy link
Member

arqunis commented Dec 7, 2018

Could you also change the #[cfg_attr(feature = "clippy", random_clippy_lint)] attributes to #[clippy::random_clippy_lint]?

src/model/id.rs Outdated Show resolved Hide resolved
@arqunis arqunis merged commit c36bd39 into serenity-rs:v0.6.x Dec 7, 2018
@Lakelezz Lakelezz deleted the update-to-rust-2018 branch December 9, 2018 16:48
arqunis pushed a commit to arqunis/serenity that referenced this pull request Apr 6, 2019
arqunis pushed a commit to arqunis/serenity that referenced this pull request Apr 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement to Serenity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants