Merge branch 'main' into joss

Author: xuanxu

Gemfile.lock

@@ -9,20 +9,20 @@ GEM
     charlock_holmes (0.7.7)
     chronic (0.10.2)
     concurrent-ruby (1.2.2)
-    connection_pool (2.4.0)
+    connection_pool (2.4.1)
     crack (0.4.5)
       rexml
     diff-lcs (1.5.0)
     dotenv (2.8.1)
-    faraday (2.7.4)
+    faraday (2.7.9)
       faraday-net_http (>= 2.0, < 3.1)
       ruby2_keywords (>= 0.0.4)
     faraday-follow_redirects (0.1.0)
       faraday (>= 2, < 3)
     faraday-net_http (3.0.2)
-    faraday-retry (2.1.0)
+    faraday-retry (2.2.0)
       faraday (~> 2.0)
-    github-linguist (7.25.0)
+    github-linguist (7.26.0)
       cgi
       charlock_holmes (~> 0.7.7)
       mini_mime (~> 1.0)
@@ -44,8 +44,8 @@ GEM
     mustermann (3.0.0)
       ruby2_keywords (~> 0.0.1)
     nio4r (2.5.9)
-    nokogiri (1.14.3)
-      mini_portile2 (~> 2.8.0)
+    nokogiri (1.15.2)
+      mini_portile2 (~> 2.8.2)
       racc (~> 1.4)
     octokit (6.1.1)
       faraday (>= 1, < 3)
@@ -54,9 +54,9 @@ GEM
       faraday (~> 2.7)
     openssl (3.1.0)
     public_suffix (5.0.1)
-    puma (6.2.2)
+    puma (6.3.0)
       nio4r (~> 2.0)
-    racc (1.6.2)
+    racc (1.7.1)
     rack (2.2.7)
     rack-protection (3.0.5)
       rack
@@ -79,7 +79,7 @@ GEM
     rspec-mocks (3.12.5)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.12.0)
-    rspec-support (3.12.0)
+    rspec-support (3.12.1)
     ruby2_keywords (0.0.5)
     rugged (1.6.3)
     sawyer (0.9.2)
@@ -91,7 +91,7 @@ GEM
       multi_json (~> 1.15)
       rexml (~> 3.2, >= 3.2.5)
       thor (~> 1.2, >= 1.2.1)
-    sidekiq (7.1.0)
+    sidekiq (7.1.2)
       concurrent-ruby (< 2)
       connection_pool (>= 2.3.0)
       rack (>= 2.2.4)
@@ -107,8 +107,8 @@ GEM
       rack-protection (= 3.0.5)
       sinatra (= 3.0.5)
       tilt (~> 2.0)
-    thor (1.2.1)
-    tilt (2.1.0)
+    thor (1.2.2)
+    tilt (2.2.0)
     webmock (3.18.1)
       addressable (>= 2.8.0)
       crack (>= 0.3.2)

app/responders/reviewers_list_responder.rb

@@ -6,7 +6,7 @@ class ReviewersListResponder < Responder
 
   def define_listening
     @event_action = "issue_comment.created"
-    @event_regex = /\A@#{bot_name} (add|remove) (\S+) (to reviewers|from reviewers|as reviewer)\.?\s*$/i
+    @event_regex = /\A@#{bot_name} (add|remove) +(\S+) +(to reviewers|from reviewers|as reviewer)\.?\s*$/i
   end
 
   def process_message(message)

spec/responders/reviewers_list_responder_spec.rb

@@ -15,13 +15,18 @@
 
     it "should define regex" do
       expect(@responder.event_regex).to match("@botsci add @arfon to reviewers")
+      expect(@responder.event_regex).to match("@botsci add   @arfon to reviewers")
+      expect(@responder.event_regex).to match("@botsci add @arfon  to reviewers")
+      expect(@responder.event_regex).to match("@botsci add   @arfon    to reviewers")
       expect(@responder.event_regex).to match("@botsci add @arfon as reviewer")
       expect(@responder.event_regex).to match("@botsci add me as reviewer")
       expect(@responder.event_regex).to match("@botsci remove me from reviewers")
       expect(@responder.event_regex).to match("@botsci remove @arfon from reviewers  ")
       expect(@responder.event_regex).to match("@botsci remove @arfon from reviewers  \r\n")
       expect(@responder.event_regex).to match("@botsci remove @arfon from reviewers  \r\n more ")
       expect(@responder.event_regex).to_not match("@botsci add to reviewers")
+      expect(@responder.event_regex).to_not match("@botsci add@arfon to reviewers")
+      expect(@responder.event_regex).to_not match("@botsci add @arfonto reviewers")
       expect(@responder.event_regex).to_not match("@botsci remove   from reviewers")
     end
   end