Update src/bin/pr-metadata-validator.rs

LonMcGregor · illicitonion · web-flow · commit 06ffe542135e · 2025-12-23T10:29:23.000Z
Co-authored-by: Daniel Wagner-Hall &lt;daniel@codeyourfuture.io&gt;
diff --git a/src/bin/pr-metadata-validator.rs b/src/bin/pr-metadata-validator.rs
@@ -265,14 +265,10 @@ async fn validate_pr(
     )
     .await
     {
-        Ok(Some(problem)) => return Ok(problem),
-        Ok(None) => (),
-        Err(e) => {
-            let _ = anyhow!(e);
-        }
+        Ok(Some(problem)) => Ok(problem),
+        Ok(None) => Ok(ValidationResult::Ok),
+        Err(err) => Err(err),
     }
-
-    Ok(ValidationResult::Ok)
 }
 
 // Check the changed files in a pull request match what is expected for that sprint task

Original file line number	Diff line number	Diff line change
`@@ -265,14 +265,10 @@ async fn validate_pr(`
`265`	`265`	`)`
`266`	`266`	`.await`
`267`	`267`	`{`
`268`		`- Ok(Some(problem)) => return Ok(problem),`
`269`		`- Ok(None) => (),`
`270`		`- Err(e) => {`
`271`		`- let _ = anyhow!(e);`
`272`		`- }`
	`268`	`+ Ok(Some(problem)) => Ok(problem),`
	`269`	`+ Ok(None) => Ok(ValidationResult::Ok),`
	`270`	`+ Err(err) => Err(err),`
`273`	`271`	`}`
`274`		`-`
`275`		`- Ok(ValidationResult::Ok)`
`276`	`272`	`}`
`277`	`273`
`278`	`274`	`// Check the changed files in a pull request match what is expected for that sprint task`