Merge pull request #50 from ehuss/dont-panic

Don't panic for non-alphanumeric reviewer
rust-lang · Mar 29, 2023 · d30d590 · d30d590
2 parents 98590d8 + 66fa049
commit d30d590
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/src/main.rs b/src/main.rs
@@ -321,12 +321,11 @@ fn parse_bors_reviewer(
             .filter(|r| !r.is_empty())
             .filter(|r| *r != "<try>")
             .inspect(|r| {
-                if !r
-                    .chars()
-                    .all(|c| c.is_alphabetic() || c.is_digit(10) || c == '-' || c == '_')
-                {
-                    panic!(
-                        "to_author for {} contained non-alphabetic characters: {:?}",
+                if !r.chars().all(|c| {
+                    c.is_alphabetic() || c.is_digit(10) || c == '-' || c == '_' || c == '='
+                }) {
+                    eprintln!(
+                        "warning: to_author for {} contained non-alphabetic characters: {:?}",
                         commit.id(),
                         r
                     );