lita-github-pinger 0.5.7 → 0.5.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0771d4f83b1f8b359099c48a25a8742a4553a6a8
-  data.tar.gz: 9ff3e9dfdd9a6d8c88125b47373360c904e1a188
+  metadata.gz: b594bf9b9431f0ad27149c953c92e38dddbf1f26
+  data.tar.gz: d78897eae20042b7b049f67e96d1f68af2bdf8a5
 SHA512:
-  metadata.gz: 0145f4acd1855ddf11564e5bb330b5763e74eba6a5a6a7ff144109f3f65c4ffbfc714cfd6a7984b4886a9a418dc8889fdcb7894637a0e5d3e55968d2b7fe4997
-  data.tar.gz: 68367a6cdeb56924894f084c20c36c41a702813207738124200bc09fba007de4db13916a9da74162c23a02cbc29e8f0fb86ef64a777ab4f83a4e0c735b30ac22
+  metadata.gz: b3bc94bb6a94c3e92a402aa473d70b9c963e5484af79a61e127ca8bd65db14576a854e39bfb5deba671d6db9093144d521609c67dbb8b1ccf903b1d6ee1ec72e
+  data.tar.gz: bae42cbe1d15873aa803031cfcd78ab2c017c0ca0b2f07329eea0a7c6af09a0ca5e780141036bf26cefc03ac59f21aee36869fd533c823c0624533b9c337da71

data/lib/lita/handlers/github_pinger.rb

@@ -34,10 +34,9 @@ module Lita
       config :engineers, type: Hash, required: true
 
       http.post("/ghping", :ghping)
-      http.post("/travisping", :travisping)
 
       def ghping(request, response)
-        puts "########## New GH PR Event! ##########"
+        puts "########## New GitHub Event! ##########"
         body = MultiJson.load(request.body)
 
         if body["comment"]
@@ -47,46 +46,38 @@ module Lita
         if body["action"] && body["action"] == "assigned"
           act_on_assign(body, response)
         end
-      end
-
-      def travisping(request, response)
-        puts "########## New Travis Event! ##########"
-        body = MultiJson.load(request.body)
 
-        committer = find_engineer(name: body["matrix"]["author_name"])
-        build_status = body["status_message"]
+        if body["state"] && body["state"] == "success"
+          act_on_build_success(body, response)
+        end
 
-        if build_status == "Passed"
-          act_on_build_success(commiter, body)
-        elsif ["Broken", "Failed"].include?(build_status)
-          act_on_build_failure(commiter, body)
+        if body["state"] && body["state"] == "failure"
+          act_on_build_failure(body, response)
         end
       end
 
-      def act_on_build_failure(commiter, body)
-        pr_url = body["repository"]["url"] + "/pull/" + body["pull_request_number"]
-        build_url = body["build_url"]
+      def act_on_build_failure(body, response)
+        commit_url = body["commit"]["html_url"]
+        committer = find_engineer(github: github["commit"]["committer"]["login"])
 
-        puts "Detected that the travis build was for PR #{pr_url}"
-        message = ":x: Your pull request failed its travis build."
-        message += "#{pr_url}\n#{build_url}"
+        puts "Detected a travis build failure for commit #{body["sha"]}"
+        message = ":x: Your commit failed some tests."
+        message += "\n#{commit_url}"
 
-        if commiter[:travis_preferences][:frequency] != "only_passes"
-          send_dm(commiter[:usernames][:slack], message)
-        end
+        return if ["off", "only_passes"].include?(commiter[:travis_preferences][:frequency])
+        send_dm(commiter[:usernames][:slack], message)
       end
 
-      def act_on_build_success(commiter, body)
-        pr_url = body["repository"]["url"] + "/pull/" + body["pull_request_number"]
-        build_url = body["build_url"]
+      def act_on_build_success(body, response)
+        commit_url = body["commit"]["html_url"]
+        committer = find_engineer(github: github["commit"]["committer"]["login"])
 
-        puts "Detected that the travis build was for PR #{pr_url}"
-        message = ":white_check_mark: Your pull request has passed its travis build."
-        message += "#{pr_url}\n#{build_url}"
+        puts "Detected a travis build success for commit #{body["sha"]}"
+        message = ":white_check_mark: Your commit has passed its travis build."
+        message += "\n#{commit_url}"
 
-        if commiter[:travis_preferences][:frequency] != "only_failures"
-          send_dm(commiter[:usernames][:slack], message)
-        end
+        return if ["off", "only_failures"].include?(commiter[:travis_preferences][:frequency])
+        send_dm(commiter[:usernames][:slack], message)
       end
 
       def act_on_assign(body, response)

data/lita-github-pinger.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-github-pinger"
-  spec.version       = "0.5.7"
+  spec.version       = "0.5.8"
   spec.authors       = ["Taylor Lapeyre"]
   spec.email         = ["taylorlapeyre@gmail.com"]
   spec.description   = "A Lita handler that detects github comment notifications and regurgitates a ping to the correct slack username."

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-github-pinger
 version: !ruby/object:Gem::Version
-  version: 0.5.7
+  version: 0.5.8
 platform: ruby
 authors:
 - Taylor Lapeyre