Merge pull request #89 from rust-lang/pa-pr

Mark-Simulacrum · web-flow · commit b7791b2189c6 · 2025-05-13T14:54:12.000+02:00
Learn from PR builds too
diff --git a/src/bin/server/worker.rs b/src/bin/server/worker.rs
@@ -171,11 +171,11 @@ impl Worker {
         if !outcome.is_passed() {
             self.report_failed(build_id, build.as_ref())?;
         }
-        if build.pr_number().is_none() && build.branch_name() == "auto" {
+        if build.pr_number().is_some() || build.branch_name() == "auto" {
             info!("learning from the log");
             self.learn(build.as_ref())?;
         } else {
-            info!("did not learn as it's not an auto build");
+            info!("did not learn as it's not an auto build or a PR build");
         }
 
         Ok(ProcessOutcome::Continue)

Original file line number	Diff line number	Diff line change
`@@ -171,11 +171,11 @@ impl Worker {`
`171`	`171`	`if !outcome.is_passed() {`
`172`	`172`	`self.report_failed(build_id, build.as_ref())?;`
`173`	`173`	`}`
`174`		`- if build.pr_number().is_none() && build.branch_name() == "auto" {`
	`174`	`+ if build.pr_number().is_some() \|\| build.branch_name() == "auto" {`
`175`	`175`	`info!("learning from the log");`
`176`	`176`	`self.learn(build.as_ref())?;`
`177`	`177`	`} else {`
`178`		`- info!("did not learn as it's not an auto build");`
	`178`	`+ info!("did not learn as it's not an auto build or a PR build");`
`179`	`179`	`}`
`180`	`180`
`181`	`181`	`Ok(ProcessOutcome::Continue)`