Fix clippy complaints

Author: saethlin

src/agent/mod.rs

@@ -13,7 +13,6 @@ use crate::utils::disk_usage::DiskUsage;
 use failure::Error;
 use rustwide::Workspace;
 use std::collections::BTreeSet;
-use std::iter::FromIterator;
 use std::ops;
 use std::sync::atomic::{AtomicBool, Ordering};
 use std::thread;

src/assets.rs

@@ -1,5 +1,5 @@
 use crate::prelude::*;
-use mime::{self, Mime};
+use mime::Mime;
 use serde::Serialize;
 use std::borrow::Cow;
 use std::collections::HashMap;

src/cli.rs

@@ -34,7 +34,7 @@ use std::time::Duration;
 pub struct Ex(String);
 
 #[derive(Debug, Clone)]
-pub struct DockerEnv(String);
+pub struct DockerEnv(#[allow(unused)] String);
 impl FromStr for Ex {
     type Err = Error;
 

src/crates/mod.rs

@@ -6,7 +6,6 @@ use crate::prelude::*;
 use cargo_metadata::PackageId;
 use percent_encoding::{percent_decode_str, utf8_percent_encode, NON_ALPHANUMERIC};
 use rustwide::Crate as RustwideCrate;
-use std::convert::TryFrom;
 use std::fmt;
 use std::path::Path;
 use std::str::FromStr;

src/db/migrations.rs

@@ -1,8 +1,5 @@
 use crate::prelude::*;
-use rand::{
-    self,
-    distributions::{Alphanumeric, DistString},
-};
+use rand::distributions::{Alphanumeric, DistString};
 use rusqlite::{Connection, Transaction};
 use std::collections::HashSet;
 
@@ -388,7 +385,7 @@ pub fn execute(db: &mut Connection) -> Fallible<()> {
     };
 
     for &(name, ref migration) in &migrations() {
-        if !executed_migrations.contains(&name.to_string()) {
+        if !executed_migrations.contains(name) {
             let t = db.transaction()?;
             match migration {
                 MigrationKind::SQL(sql) => t.execute_batch(sql),

src/report/mod.rs

@@ -8,12 +8,11 @@ use crate::results::{EncodedLog, EncodingType, FailureReason, ReadResults, TestR
 use crate::toolchain::Toolchain;
 use crate::utils;
 use crates_index::GitIndex;
-use mime::{self, Mime};
+use mime::Mime;
 use percent_encoding::{utf8_percent_encode, AsciiSet};
 use std::borrow::Cow;
 #[cfg(test)]
 use std::collections::HashMap;
-use std::convert::AsRef;
 use std::fmt::{self, Display};
 use std::fs;
 use std::path::{Path, PathBuf};

src/runner/test.rs

@@ -13,7 +13,6 @@ use rustwide::cmd::{CommandError, ProcessLinesActions, SandboxBuilder};
 use rustwide::logging::LogStorage;
 use rustwide::{Build, PrepareError};
 use std::collections::{BTreeSet, HashMap, HashSet};
-use std::convert::TryFrom;
 
 fn failure_reason(err: &Error) -> FailureReason {
     for cause in err.iter_chain() {

src/server/auth.rs

@@ -7,7 +7,7 @@ use regex::Regex;
 use rust_team_data::v1 as team_data;
 use std::collections::{HashMap, HashSet};
 use std::sync::{Arc, RwLock};
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 lazy_static! {
     static ref GIT_REVISION_RE: Regex =

src/server/mod.rs

@@ -17,12 +17,12 @@ use crate::server::agents::Agents;
 use crate::server::auth::ACL;
 use crate::server::github::{GitHub, GitHubApi};
 use crate::server::tokens::{BotTokens, Tokens};
-use http::{self, header::HeaderValue, Response};
+use http::{header::HeaderValue, Response};
 use hyper::Body;
 use metrics::Metrics;
 use std::net::SocketAddr;
 use std::sync::{Arc, Mutex};
-use warp::{self, Filter};
+use warp::Filter;
 
 lazy_static! {
     static ref SERVER_HEADER: String =

src/server/routes/agent.rs

@@ -14,7 +14,7 @@ use hyper::Body;
 use std::collections::HashMap;
 use std::sync::{Arc, Condvar, Mutex};
 use std::time::Instant;
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 #[derive(Deserialize)]
 #[serde(rename_all = "kebab-case")]

src/server/routes/metrics.rs

@@ -5,7 +5,7 @@ use http::{Response, StatusCode};
 use hyper::Body;
 use prometheus::{Encoder, TextEncoder};
 use std::sync::Arc;
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 pub fn routes(
     data: Arc<Data>,

src/server/routes/ui/mod.rs

@@ -6,7 +6,7 @@ use http::{Response, StatusCode};
 use hyper::Body;
 use serde::Serialize;
 use std::sync::Arc;
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 mod agents;
 mod experiments;

src/server/routes/webhooks/args.rs

@@ -1,6 +1,5 @@
 use crate::experiments::{Assignee, CapLints, DeferredCrateSelect, Mode};
 use crate::toolchain::Toolchain;
-use failure::{self, Fallible};
 
 #[derive(Debug, thiserror::Error)]
 #[cfg_attr(test, derive(PartialEq, Eq))]

src/server/routes/webhooks/mod.rs

@@ -12,7 +12,7 @@ use http::{HeaderMap, Response, StatusCode};
 use hyper::Body;
 use std::str::FromStr;
 use std::sync::Arc;
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 fn process_webhook(
     payload: &[u8],

src/utils/mod.rs

@@ -1,5 +1,5 @@
 use crate::prelude::*;
-use failure::{AsFail, Backtrace, Fail};
+use failure::{AsFail, Backtrace};
 use percent_encoding::{AsciiSet, CONTROLS};
 use std::any::Any;
 use std::fmt;

src/utils/serialize.rs

@@ -1,5 +1,4 @@
 use serde::ser::{Serialize, SerializeSeq, Serializer};
-use std::iter::IntoIterator;
 
 pub fn to_vec<S, T>(data: T, serializer: S) -> Result<S::Ok, S::Error>
 where