mass-rebuilder: It's always nixpkgs

NixOS · Dec 29, 2024 · 5ed4d64 · 5ed4d64
1 parent 4c2ad4f
commit 5ed4d64
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 65 deletions.
diff --git a/ofborg/src/message/evaluationjob.rs b/ofborg/src/message/evaluationjob.rs
@@ -11,12 +11,6 @@ pub struct EvaluationJob {
     pub pr: Pr,
 }
 
-impl EvaluationJob {
-    pub fn is_nixpkgs(&self) -> bool {
-        self.repo.name == "nixpkgs"
-    }
-}
-
 pub struct Actions {}
 
 impl Actions {

diff --git a/ofborg/src/tasks/eval/generic.rs b/ofborg/src/tasks/eval/generic.rs
diff --git a/ofborg/src/tasks/evaluate.rs b/ofborg/src/tasks/evaluate.rs
@@ -291,19 +291,15 @@ impl<'a, E: stats::SysEvents + 'static> OneEval<'a, E> {
             }
         };
 
-        let mut evaluation_strategy: Box<dyn eval::EvaluationStrategy> = if job.is_nixpkgs() {
-            Box::new(eval::NixpkgsStrategy::new(
-                job,
-                &pull,
-                &issue,
-                &issue_ref,
-                &repo,
-                &self.gists,
-                self.nix.clone(),
-            ))
-        } else {
-            Box::new(eval::GenericStrategy::new())
-        };
+        let mut evaluation_strategy = eval::NixpkgsStrategy::new(
+            job,
+            &pull,
+            &issue,
+            &issue_ref,
+            &repo,
+            &self.gists,
+            self.nix.clone(),
+        );
 
         let prefix = get_prefix(repo.statuses(), &job.pr.head_sha)?;