diff --git a/ofborg/src/lib.rs b/ofborg/src/lib.rs index 87ea893..43dca6b 100644 --- a/ofborg/src/lib.rs +++ b/ofborg/src/lib.rs @@ -1,4 +1,9 @@ #![recursion_limit = "512"] +// Replacing .map(|arch| arch.to_string()) +// with .map(systems::System::to_string) +// +// seems much less clear and I just don't like it :) +#![allow(clippy::redundant_closure)] #[macro_use] extern crate serde_derive; diff --git a/ofborg/src/tasks/evaluate.rs b/ofborg/src/tasks/evaluate.rs index 6a82eb3..3b3f236 100644 --- a/ofborg/src/tasks/evaluate.rs +++ b/ofborg/src/tasks/evaluate.rs @@ -70,8 +70,8 @@ impl EvaluationWorker { Err(()) } Err(eval::Error::FailWithGist(msg, filename, content)) => { - status.set_with_description(&msg, hubcaps::statuses::State::Failure); status.set_url(make_gist(&gists, &filename, Some("".to_owned()), content)); + status.set_with_description(&msg, hubcaps::statuses::State::Failure); Err(()) } }