danderson/ofborg
0
0
Fork 0
forked from the-distro/ofborg
Code Pull requests Activity

treewide: replace {:#?} with {:?}

Browse source 
Cleans up logging output so that things that belong together stay
together.
This commit is contained in:
Cole Helbling 2020-04-05 12:02:56 -07:00
parent e26497e49d
commit 5c1a85ca31
No known key found for this signature in database
GPG key ID: B37E0F2371016A4C
4 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
ofborg/src/config.rs
View file

@ -197,11 +197,11 @@ impl GithubAppVendingMachine {
match lookup_gh.app().find_repo_installation(owner, repo) { match lookup_gh.app().find_repo_installation(owner, repo) {
Ok(install_id) => { Ok(install_id) => {
debug!("Received install ID {:#?}", install_id); debug!("Received install ID {:?}", install_id);
Some(install_id.id) Some(install_id.id)
} }
Err(e) => { Err(e) => {
warn!("Error during install ID lookup: {:#?}", e); warn!("Error during install ID lookup: {:?}", e);
None None
} }
} }

2
ofborg/src/nixenv.rs
View file

@ -155,7 +155,7 @@ impl Error {
} }
} }
Error::UncleanEvaluation(warnings) => { Error::UncleanEvaluation(warnings) => {
format!("nix-env did not evaluate cleanly:\n {:#?}", warnings) format!("nix-env did not evaluate cleanly:\n {:?}", warnings)
} }
Error::StatsParse(mut fd, seek, parse_err) => { Error::StatsParse(mut fd, seek, parse_err) => {
let mut buffer = Vec::new(); let mut buffer = Vec::new();

2
ofborg/src/tasks/eval/nixpkgs.rs
View file

@ -575,7 +575,7 @@ fn request_reviews(maint: &maintainers::ImpactedMaintainers, pull: &hubcaps::pul
team_reviewers: vec![], team_reviewers: vec![],
}) })
{ {
println!("Failure requesting a review from {}: {:#?}", maintainer, e,); println!("Failure requesting a review from {}: {:?}", maintainer, e,);
} }
} }
} }

10
ofborg/src/tasks/evaluate.rs
View file

@ -211,7 +211,7 @@ impl<'a, E: stats::SysEvents + 'static> OneEval<'a, E> {
match eval_result { match eval_result {
EvalWorkerError::CommitStatusWrite(e) => { EvalWorkerError::CommitStatusWrite(e) => {
eprintln!( eprintln!(
"Failed to write commit status, got error: {:#?}, marking internal error", "Failed to write commit status, got error: {:?}, marking internal error",
e e
); );
let issue_ref = self.repo.issue(self.job.pr.number); let issue_ref = self.repo.issue(self.job.pr.number);
@ -219,7 +219,7 @@ impl<'a, E: stats::SysEvents + 'static> OneEval<'a, E> {
} }
EvalWorkerError::EvalError(eval::Error::CommitStatusWrite(e)) => { EvalWorkerError::EvalError(eval::Error::CommitStatusWrite(e)) => {
eprintln!( eprintln!(
"Failed to write commit status, got error: {:#?}, marking internal error", "Failed to write commit status, got error: {:?}, marking internal error",
e e
); );
let issue_ref = self.repo.issue(self.job.pr.number); let issue_ref = self.repo.issue(self.job.pr.number);
@ -228,7 +228,7 @@ impl<'a, E: stats::SysEvents + 'static> OneEval<'a, E> {
EvalWorkerError::EvalError(eval::Error::Fail(msg)) => { EvalWorkerError::EvalError(eval::Error::Fail(msg)) => {
self.update_status(msg.clone(), None, hubcaps::statuses::State::Failure) self.update_status(msg.clone(), None, hubcaps::statuses::State::Failure)
.unwrap_or_else(|e| { .unwrap_or_else(|e| {
panic!("Failed to set plain status: {}; e: {:#?}", msg, e); panic!("Failed to set plain status: {}; e: {:?}", msg, e);
}); });
} }
EvalWorkerError::EvalError(eval::Error::FailWithGist(msg, filename, content)) => { EvalWorkerError::EvalError(eval::Error::FailWithGist(msg, filename, content)) => {
@ -239,7 +239,7 @@ impl<'a, E: stats::SysEvents + 'static> OneEval<'a, E> {
) )
.unwrap_or_else(|e| { .unwrap_or_else(|e| {
panic!( panic!(
"Failed to set status with a gist: {}, {}, {}; e: {:#?}", "Failed to set status with a gist: {}, {}, {}; e: {:?}",
msg, filename, content, e msg, filename, content, e
); );
}); });
@ -469,7 +469,7 @@ fn schedule_builds(
) -> Vec<worker::Action> { ) -> Vec<worker::Action> {
let mut response = vec![]; let mut response = vec![];
info!( info!(
"Scheduling build jobs {:#?} on arches {:#?}", "Scheduling build jobs {:?} on arches {:?}",
builds, auto_schedule_build_archs builds, auto_schedule_build_archs
); );
for buildjob in builds { for buildjob in builds {