Merge pull request #1735 from ehuss/rr-no-config

Review requested: don't error if not configured
rust-lang · Oct 22, 2023 · d3eb488 · d3eb488
2 parents 75f6785 + 74f2ee1
commit d3eb488
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/handlers/review_requested.rs b/src/handlers/review_requested.rs
@@ -7,10 +7,14 @@ pub(crate) struct ReviewRequestedInput {}
 pub(crate) async fn parse_input(
     _ctx: &Context,
     event: &IssuesEvent,
-    _config: Option<&ReviewRequestedConfig>,
+    config: Option<&ReviewRequestedConfig>,
 ) -> Result<Option<ReviewRequestedInput>, String> {
     // PR author requests a review from one of the assignees
 
+    if config.is_none() {
+        return Ok(None);
+    }
+
     let IssuesAction::ReviewRequested { requested_reviewer } = &event.action else {
         return Ok(None);
     };