github-nippou 2.0.0.beta3 → 2.0.0.beta4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 395e6dc886fa422488530971c407766a8e9f40a2
-  data.tar.gz: d28cc82a0bf38888a818a4ad8b98bc42e9d0896a
+  metadata.gz: e04ff32f953611852d55ee08062486083beda4c6
+  data.tar.gz: 678f97c55b3b566e561c97b903fac0f517edea6e
 SHA512:
-  metadata.gz: d37a77092bcb34e48fc2351a4165e2a7adf18e506cd057dd7057faadc98ad97c2dd3c4b27ffe477c4b83662cfcc5cddb40a951c30eee2473fa80c4c670956e97
-  data.tar.gz: c1a5a7c61ffa5b14a4f93cf148c2adf5997ee4b602db560d83536451b9abf5b049dd28bcd8f1cd67ae4873e94d6103c6bdd779b8722751584897ad917ced8283
+  metadata.gz: 73d1e69a9dd7ca65092a4e6b58d8194fcb1057c4b024eb69e3cf459cb1dc3a1fc5620159b6e25ea79368aeed818ccee5eb0343bb7c32870c30a4255da787a6f4
+  data.tar.gz: db8c91944b4d351c16c75263e89d66a5c7a92ef17acffd61edc576100cc748d336be022426c738d13011e589e82db5b95829d8bb0964e2af34bb30567ca86183

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    github-nippou (2.0.0.beta3)
+    github-nippou (2.0.0.beta4)
       activesupport
       octokit
       thor

data/README.md

@@ -33,6 +33,7 @@ $ github-nippou help
 Commands:
   github-nippou help [COMMAND]  # Describe available commands or one specific command
   github-nippou list            # Displays today's GitHub events formatted for Nippou
+  github-nippou version         # Displays version
 
 Options:
   s, [--since-date=SINCE_DATE]  # Retrieves GitHub user_events since the date

data/lib/github/nippou.rb

@@ -1,3 +1,6 @@
+require 'github/nippou/concerns/sawyer_resource_github'
+require 'github/nippou/concerns/string_markdown'
+
 require 'github/nippou/commands'
 require 'github/nippou/user_events'
 require 'github/nippou/version'

data/lib/github/nippou/commands.rb

@@ -1,17 +1,10 @@
 require 'thor'
 
-module StringExMarkdown
-  refine String do
-    def markdown_escape
-      self.gsub(/([`<>])/, '\\\\\1')
-    end
-  end
-end
-
 module Github
   module Nippou
     class Commands < Thor
-      using StringExMarkdown
+      using SawyerResourceGithub
+      using StringMarkdown
 
       default_task :list
       class_option :since_date, type: :string,
@@ -23,11 +16,13 @@ module Github
 
       desc 'list', "Displays today's GitHub events formatted for Nippou"
       def list
-        nippous.each do |url, detail|
-          line = "* [#{detail[:title]} - #{detail[:repo_basename]}](#{url}) by #{detail[:username]}"
-          if detail[:merged]
+        user_events.each do |user_event|
+          issue = user_event.issue(client)
+          line = "* [%s - %s](%s) by %s" %
+                 [issue.title.markdown_escape, user_event.repo.name, user_event.html_url, issue.user.login]
+          if issue.merged
             line << ' **merged!**'
-          elsif detail[:state] == 'closed'
+          elsif issue.state == 'closed'
             line << ' **closed!**'
           end
           puts line
@@ -41,27 +36,10 @@ module Github
 
       private
 
-      def nippous
-        result = {}
-
-        user_events.each do |user_event|
-          case user_event.type
-          when 'IssuesEvent', 'IssueCommentEvent'
-            issue = user_event.payload.issue
-            result[issue.html_url] ||= hash_for_issue(user_event.repo.name, issue.number)
-          when 'PullRequestEvent', 'PullRequestReviewCommentEvent'
-            pr = user_event.payload.pull_request
-            result[pr.html_url] ||= hash_for_pr(user_event.repo.name, pr.number)
-          end
-        end
-
-        result.sort
-      end
-
       def user_events
         @user_events ||= UserEvents.new(
           client, user, options[:since_date], options[:until_date]
-        ).retrieve
+        ).collect
       end
 
       def client
@@ -106,30 +84,6 @@ MESSAGE
             exit!
           end
       end
-
-      def hash_for_issue(repo_name, issue_number)
-        issue = client.issue(repo_name, issue_number)
-
-        {
-          title: issue.title.markdown_escape,
-          repo_basename: repo_name,
-          username: issue.user.login,
-          merged: issue.merged,
-          state: issue.state,
-        }
-      end
-
-      def hash_for_pr(repo_name, pr_number)
-        pr = client.pull_request(repo_name, pr_number)
-
-        {
-          title: pr.title.markdown_escape,
-          repo_basename: repo_name,
-          username: pr.user.login,
-          merged: pr.merged,
-          state: pr.state,
-        }
-      end
     end
   end
 end

data/lib/github/nippou/concerns/sawyer_resource_github.rb

@@ -0,0 +1,33 @@
+require 'octokit'
+
+module SawyerResourceGithub
+  refine Sawyer::Resource do
+    def issue(client)
+      case
+      when self.issue?
+        client.issue(self.repo.name, self.payload.issue.number)
+      when self.pull_request?
+        client.pull_request(self.repo.name, self.payload.pull_request.number)
+      end
+    end
+
+    def html_url
+      case
+      when self.issue?
+        self.payload.issue.html_url
+      when self.pull_request?
+        self.payload.pull_request.html_url
+      end
+    end
+
+    def issue?
+      self.type == 'IssuesEvent' ||
+        self.type == 'IssueCommentEvent'
+    end
+
+    def pull_request?
+      self.type == 'PullRequestEvent' ||
+        self.type == 'PullRequestReviewCommentEvent'
+    end
+  end
+end

data/lib/github/nippou/concerns/string_markdown.rb

@@ -0,0 +1,7 @@
+module StringMarkdown
+  refine String do
+    def markdown_escape
+      self.gsub(/([`<>])/, '\\\\\1')
+    end
+  end
+end

data/lib/github/nippou/user_events.rb

@@ -4,13 +4,22 @@ require 'octokit'
 module Github
   module Nippou
     class UserEvents
+      using SawyerResourceGithub
+
       def initialize(client, user, since_date, until_date)
         @client = client
         @user = user
-        @since_date = Time.parse(since_date).beginning_of_day
-        @until_date = Time.parse(until_date).end_of_day
+        @since_time = Time.parse(since_date).beginning_of_day
+        until_time = Time.parse(until_date).end_of_day
+        @range = @since_time..until_time
+      end
+
+      def collect
+        uniq(sort(filter(retrieve)))
       end
 
+      private
+
       def retrieve
         user_events = @client.user_events(@user, per_page: 100)
         last_response = @client.last_response
@@ -23,15 +32,31 @@ module Github
         user_events.select { |user_event| in_range?(user_event) }
       end
 
-      private
-
       def continue?(last_response, user_events)
         last_response.rels[:next] &&
-          user_events.last.created_at.getlocal >= @since_date
+          user_events.last.created_at.getlocal >= @since_time
       end
 
       def in_range?(user_event)
-        (@since_date..@until_date).include?(user_event.created_at.getlocal)
+        @range.include?(user_event.created_at.getlocal)
+      end
+
+      def filter(user_events)
+        user_events.select do |user_event|
+          user_event.issue? || user_event.pull_request?
+        end
+      end
+
+      def sort(user_events)
+        user_events.sort do |a, b|
+          a.html_url <=> b.html_url
+        end
+      end
+
+      def uniq(user_events)
+        user_events.uniq do |user_event|
+          user_event.html_url
+        end
       end
     end
   end

data/lib/github/nippou/version.rb

@@ -1,5 +1,5 @@
 module Github
   module Nippou
-    VERSION = '2.0.0.beta3'
+    VERSION = '2.0.0.beta4'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: github-nippou
 version: !ruby/object:Gem::Version
-  version: 2.0.0.beta3
+  version: 2.0.0.beta4
 platform: ruby
 authors:
 - Takashi Masuda
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-26 00:00:00.000000000 Z
+date: 2016-03-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -100,6 +100,8 @@ files:
 - github-nippou.gemspec
 - lib/github/nippou.rb
 - lib/github/nippou/commands.rb
+- lib/github/nippou/concerns/sawyer_resource_github.rb
+- lib/github/nippou/concerns/string_markdown.rb
 - lib/github/nippou/user_events.rb
 - lib/github/nippou/version.rb
 homepage: https://github.com/masutaka/github-nippou