Merge pull request #2038 from Urgau/assign-allow-empty-owners-table

Allow empty `[assign.owners]`

Author: Kobzol

src/handlers/assign.rs

@@ -130,11 +130,7 @@ pub(super) async fn parse_input(
     event: &IssuesEvent,
     config: Option<&AssignConfig>,
 ) -> Result<Option<AssignInput>, String> {
-    let config = match config {
-        Some(config) => config,
-        None => return Ok(None),
-    };
-    if config.owners.is_empty() || !event.issue.is_pr() {
+    if config.is_none() || !event.issue.is_pr() {
         return Ok(None);
     }
 

src/handlers/assign/tests/tests_from_diff.rs

@@ -144,3 +144,10 @@ fn basic_gitignore_pattern() {
     let diff = make_fake_diff(&[("src/librustdoc/html/static/js/settings.js", 10, 1)]);
     test_from_diff(&diff, config, &["javascript-reviewers"]);
 }
+
+#[test]
+fn empty_owners_table() {
+    let config = toml::toml!([owners]);
+    let diff = make_fake_diff(&[("src.js", 10, 1)]);
+    test_from_diff(&diff, config, &[]);
+}