From 76c57fb495866fc711d3cc87ba31c0e6466139a8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Na=C3=AFm=20Favier?= Date: Fri, 14 Oct 2022 16:12:02 +0200 Subject: [PATCH] Revert "ofborg: post a finished check if evaluation starts" This reverts commit 16355e10b4e4519c2e07915c6142d8039cfe1cc3. --- ofborg/src/tasks/evaluate.rs | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/ofborg/src/tasks/evaluate.rs b/ofborg/src/tasks/evaluate.rs index d9b1da0..223ab47 100644 --- a/ofborg/src/tasks/evaluate.rs +++ b/ofborg/src/tasks/evaluate.rs @@ -31,23 +31,6 @@ pub struct EvaluationWorker { events: E, } -/// Creates a finished github check, indicating that the ofborg eval started. -/// This is a workaround github REST api, that only exposes finished external checks. -/// We use the check status in our "Waiting for github action" -fn post_eval_started_status( - api: hubcaps::statuses::Statuses, - commit: String, -) -> Result<(), EvalWorkerError> { - Ok(CommitStatus::new( - api, - commit, - "ofborg-eval-started".to_owned(), - "Evaluation started".to_owned(), - None, - ) - .set(hubcaps::statuses::State::Success)?) -} - impl EvaluationWorker { #[allow(clippy::too_many_arguments)] pub fn new( @@ -330,8 +313,6 @@ impl<'a, E: stats::SysEvents + 'static> OneEval<'a, E> { overall_status.set_with_description("Starting", hubcaps::statuses::State::Pending)?; - post_eval_started_status(repo.statuses(), job.pr.head_sha.clone())?; - evaluation_strategy.pre_clone()?; let project = self