build-buddy 1.8.0 → 1.8.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bc533f6854862178451a21df0a4d84ae2061990b
-  data.tar.gz: be19e66a1a5dee3caf7dc857cf8960582331775a
+  metadata.gz: 656443f17239c6954ec716ef56dce64a6ad0a33b
+  data.tar.gz: bf937f8268594356e0bfc79bcbb6163a4134e386
 SHA512:
-  metadata.gz: 462ec0136b74bafc5a3a93f8c1914ad1f6b39a2a7a90b34fdea196ee180e57e3a7c63a3e4b42762881cc791fea3c773da9f72e5e43600afd247abdd3199b5883
-  data.tar.gz: 8ba1d443cc60f29858b6dc5fa27b358c7d4b74a119ea46b1939a31afff428bc446d5c7345ff25cba07416974619c843ab84a3a165efe5c5e886f27741904df54
+  metadata.gz: a04a74c2e86297a87a5f0ec98ede406e05456073479d661edd298c90df868ffc896b5b1aa4078f7e8858a2df6d5d3286434c8e60f9eacfcb3ab80db5af269994
+  data.tar.gz: 0f45e9e4bb55b51107369dad1555f3074dd6b04273c24721f72cf8b12b7c4b55cc089bb0174f85c7cf8b0f098c26dbc8e364d1fb9a676dff23eca3702820b1a8

data/lib/build_buddy.rb

@@ -9,5 +9,5 @@ require 'build_buddy/recorder'
 require 'build_buddy/build_data'
 
 module BuildBuddy
-  VERSION = "1.8.0"
+  VERSION = "1.8.4"
 end

data/lib/build_buddy/gitter.rb

@@ -14,7 +14,7 @@ module BuildBuddy
       info "Connected to Github"
     end
 
-    def set_status(repo_full_name, repo_sha, status, description, target_url)
+    def set_status(repo_full_name, repo_sha, state, description, target_url)
       options = {
           :description => description.length > 140 ? "#{description[0..136]}..." : description,
           :context => 'build-buddy'
@@ -22,7 +22,7 @@ module BuildBuddy
       unless target_url.nil?
         options[:target_url] = target_url
       end
-      @gh_client.create_status(repo_full_name, repo_sha, status.to_s, options)
+      @gh_client.create_status(repo_full_name, repo_sha, state.to_s, options)
     end
   end
 end

data/lib/build_buddy/server.rb

@@ -27,8 +27,8 @@ module BuildBuddy
                 request.respond 500, "Signatures didn't match!"
               else
                 payload = JSON.parse(payload_text)
+                action = payload['action']
                 pull_request = payload['pull_request']
-                pull_request_action = pull_request['action']
 
                 case action
                 when 'opened', 'reopened', 'synchronize'

data/lib/build_buddy/slacker.rb

@@ -269,15 +269,15 @@ Ask me `what happened` to get a list of recent builds and log files and `what op
     def notify_channel(build_data)
       status_message = build_data.termination_type == :killed ? "was stopped" : build_data.exit_code != 0 ? "failed" : "succeeded"
       status_message += '. '
+      log_url = Config.server_base_uri + '/log/' + build_data._id.to_s
 
       if build_data.type == :pull_request
-        message = "The buddy build #{status_message}"
-        git_status = (build_data.termination_type == :killed ? :failure : build_data.exit_code != 0 ? :error : :success)
-        git_url = (git_status == :error ? "#{Config.server_base_uri + '/log/' + build_data._id.to_s}" : nil)
-        Celluloid::Actor[:gitter].async.set_status(build_data.repo_full_name, build_data.repo_sha, git_status, git_message, git_url)
+        message = "Build #{status_message}"
+        git_state = (build_data.termination_type == :killed ? :failure : build_data.exit_code != 0 ? :error : :success)
+        Celluloid::Actor[:gitter].async.set_status(build_data.repo_full_name, build_data.repo_sha, git_state, message, log_url)
         info "Pull request build #{status_message}"
       else
-        status_message += "Log file at #{Config.server_base_uri + '/log/' + build_data._id.to_s}."
+        status_message += "Log file at #{log_url}."
         if build_data.type == :master
           message = "A build of the `master` branch #{status_message}"
           info "`master` branch build #{status_message}"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: build-buddy
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.8.4
 platform: ruby
 authors:
 - John Lyon-smith
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-08 00:00:00.000000000 Z
+date: 2016-04-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: timers