lita-github_pr_list 0.0.15 → 0.0.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ad7d7dbd86f8b5b0ee27cbd6b1f3f38dd3e2c7f3
-  data.tar.gz: 1e3bfd2c9b477c49b3807b0c388018c9233a9bc5
+  metadata.gz: cdf9713e32606d90c856f4cfa8e76dce822eaa6f
+  data.tar.gz: 08cb5fb6c22cb47f1e9d6df233d544088a5b14ac
 SHA512:
-  metadata.gz: fe9cdc0d3e28a3179bb51c7335ea7ca75db29f06acd1a46b79c0c09bbeed6075b524bfccc3e62dff5b846b9210229d97b2e4b56cb08bb90a06f9d3c5bac07fcc
-  data.tar.gz: 262da8e81bab986a5347a1f517cf190e672422e1c9e7c898f782f2828cb9c3ce0b9dfa3b1d682e9a4eafd06536bce74e506b13afc4c3a53474587c6b7cf5f967
+  metadata.gz: 1bdbac47ac8daeefa1e75e3c384369039346a0b0a90c7d1e6efe594aa4f29ff0f7d74422ff0c1ca4c0b530a4b3a09ceb7cb558f17c472a06df5d57c552d2cc06
+  data.tar.gz: 853e48b9c0d629a983acfc14d6b9afdcf8133f4e58a724451acc376a934e9d04ac03880fa2db3538d1f7bc9efe28097117dbfcc814f760189d849e69930d9c93

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## 2014-11-04
+
+Bug Fixes:
+
+* Adding notification sent to Hipchat, opened status only. ([#32][], [@darkodosenovic][])
+
+
 ## 2014-10-09
 
 Bug Fixes:
@@ -20,5 +27,7 @@ Features:
 [#28]: https://github.com/amaabca/lita-github_pr_list/issues/28
 [#29]: https://github.com/amaabca/lita-github_pr_list/issues/29
 [#30]: https://github.com/amaabca/lita-github_pr_list/issues/30
+[#32]: https://github.com/amaabca/lita-github_pr_list/issues/32
 [@MathieuGilbert]: https://github.com/MathieuGilbert
+[@darkodosenovic]: https://github.com/darkodosenovic
 [@rubene]: https://github.com/rubene

data/lib/lita/github_pr_list/pull_request_open_message_hook.rb

@@ -1,9 +1,10 @@
 module Lita
   module GithubPrList
     class PullRequestOpenMessageHook
-      attr_accessor :response, :request, :redis, :payload, :pull_request_owner, :title, :repo_name
+      attr_accessor :response, :request, :redis, :payload, :pull_request_owner, :title, :repo_name, :pull_request_status, :pull_request_html_url, :statuses
 
       def initialize(params = {})
+        self.statuses = %w(opened)
         self.response = params.fetch(:response, nil)
         self.request = params.fetch(:request, nil)
         self.redis = params.fetch(:redis, nil)
@@ -12,13 +13,17 @@ module Lita
 
         # https://developer.github.com/v3/activity/events/types/#pullrequestevent
         self.payload = JSON.parse(request.body.read)
+        self.pull_request_status = payload["action"]
         self.pull_request_owner = redis.get("alias:#{payload["pull_request"]["user"]["login"]}") || payload["pull_request"]["user"]["login"]
         self.title = payload["pull_request"]["title"]
         self.repo_name = payload["pull_request"]["head"]["repo"]["full_name"]
+        self.pull_request_html_url = payload["pull_request"]["html_url"]
       end
 
       def message
-        "@#{pull_request_owner} opened pull request: '#{title}' in '#{repo_name}'"
+        if statuses.include? pull_request_status
+          "@#{pull_request_owner} #{pull_request_status} pull request: '#{title}' in '#{repo_name}'. #{pull_request_html_url}'"
+        end
       end
     end
   end

data/lib/lita/github_pr_list/version.rb

@@ -1,5 +1,5 @@
 module Lita
   module GithubPrList
-    VERSION = "0.0.15"
+    VERSION = "0.0.16"
   end
 end

data/spec/lita/handlers/open_pull_request_message_hook_spec.rb

@@ -15,6 +15,6 @@ describe Lita::Handlers::GithubPrList, lita_handler: true do
     github_handler = Lita::Handlers::GithubPrList.new
     github_handler.pull_request_open_message_hook(request, response)
 
-    expect(replies.last).to include("@baxterthehacker opened pull request: 'Update the README with new information' in 'baxterthehacker/public-repo'")
+    expect(replies.last).to include("@baxterthehacker opened pull request: 'Update the README with new information' in 'baxterthehacker/public-repo'. https://github.com/baxterthehacker/public-repo/pull/48")
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-github_pr_list
 version: !ruby/object:Gem::Version
-  version: 0.0.15
+  version: 0.0.16
 platform: ruby
 authors:
 - Michael van den Beuken
@@ -13,7 +13,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-03 00:00:00.000000000 Z
+date: 2014-11-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita