sync_issues 0.7.0 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 99ef37bc236b9d418bc2026635fd15cb0257e668
-  data.tar.gz: 6fdfee2a75ffff873f3071892b9f6bcc09121333
+  metadata.gz: 79925614f29788d3b54461146e6b224d5563b341
+  data.tar.gz: 5e47b0f0f9425ac917c31117442a3449757c8c13
 SHA512:
-  metadata.gz: a0f5d617d4924d1c239090b6b6776bc8a4f3ed9c44fb292724ea4c180f502447d5358ff34003b28629d80711b86af8abb892ac6ac726a3474c0f853780a278bb
-  data.tar.gz: 1059944a2820a11895d67771c11619f642a1aba981d1d902c3bb871584056fcaf4d668892a7777c78053549187eadc6413cd47538d4599b8371831c16581dd0a
+  metadata.gz: 10dda1d05a6f9fd0d6da1bb867cf6a096568bdbefcf557247a8c3c0eca6e958f28ea1d0c7943c802329f616571536074201c770a2e10247315c3a5cf33c59998
+  data.tar.gz: 3e7eb16a7d7fba78b80c1ae666520205d0a103ffc89b64748185624426b826e94d0066d8a0bce857de69089c69d721cd307d0edd46a1ea5cf1cf5449f825410a

data/README.md

@@ -62,9 +62,11 @@ The frontmatter of an issue file can contain the following attributes:
 
 * __title__: (required) Used as the title of the issue on GitHub. The title is
   used as the unique key when syncing updated tasks with existing issues.
-* __assignee__: (optional) Assign or reassign the issue to the github username
-  specified. Existing assignee will not be removed on sync if the field is not
-  provided.
+
+* __assignees__: (optional) Assign or reassign the issue to the github
+  usernames specified. Existing assignees will not be removed on sync if the
+  field is not provided.
+
 * __labels__: (optional) When provided and the issue does not have any labels
   this list of labels will be added to the issue. Labels will be dynamically
   created with the default grey color if they don't already exist on the

data/lib/sync_issues/comparison.rb

@@ -3,16 +3,18 @@ require_relative 'error'
 module SyncIssues
   # Comparison represents differences between Issues (local and GitHub)
   class Comparison
-    attr_reader :assignee, :changed, :content, :labels, :title
+    attr_reader :assignees, :changed, :content, :labels, :title
 
     def initialize(issue, github_issue, reset_labels: false,
-                   sync_assignee: true, sync_labels: true)
+                   sync_assignees: true, sync_labels: true)
       @changed = []
-      @assignee = github_issue.assignee && github_issue.assignee.login
+      @assignees = github_issue.assignees.map do |assignee|
+        assignee.login
+      end.sort
       @content = github_issue.body
       @labels = github_issue.labels.map { |label| label[:name] }
       @title = github_issue.title
-      compare(issue, reset_labels, sync_assignee, sync_labels)
+      compare(issue, reset_labels, sync_assignees, sync_labels)
     end
 
     def changed?
@@ -21,10 +23,10 @@ module SyncIssues
 
     private
 
-    def compare(issue, reset_labels, sync_assignee, sync_labels)
-      if sync_assignee && issue.assignee != @assignee
-        @changed << 'assignee'
-        @assignee = issue.assignee
+    def compare(issue, reset_labels, sync_assignees, sync_labels)
+      if sync_assignees && update_assignees?(issue)
+        @changed << 'assignees'
+        @assignees = issue.assignees
       end
       if sync_labels && update_label?(issue, reset_labels)
         @changed << 'labels'
@@ -44,11 +46,20 @@ module SyncIssues
       first.gsub(/\[x\]/, '[ ]') == second.gsub(/\[x\]/, '[ ]')
     end
 
+    def assignees_match?(new_assignees)
+      # Assignee uniqueness is not case-sensitive.
+      new_assignees.map(&:downcase) == @assignees.map(&:downcase)
+    end
+
     def labels_match?(new_labels)
       # Label uniqueness is not case-sensitive.
       new_labels.map(&:downcase) == @labels.map(&:downcase)
     end
 
+    def update_assignees?(issue)
+      !issue.assignees.nil? && !assignees_match?(issue.assignees)
+    end
+
     def update_label?(issue, reset_labels)
       !issue.labels.nil? && (reset_labels && !labels_match?(issue.labels) ||
                              @labels.size == 0)

data/lib/sync_issues/github.rb

@@ -12,9 +12,9 @@ module SyncIssues
       @client.auto_paginate = true
     end
 
-    def create_issue(repository, issue, add_assignee, add_labels)
+    def create_issue(repository, issue, add_assignees, add_labels)
       kwargs = {}
-      kwargs[:assignee] = issue.assignee if add_assignee
+      kwargs[:assignee] = issue.assignees[0] if add_assignees
       kwargs[:labels] = issue.labels if add_labels
       @client.create_issue(repository.full_name, issue.title, issue.content,
                            **kwargs)
@@ -39,8 +39,12 @@ module SyncIssues
     def update_issue(repository, issue_number, comparison)
       @client.update_issue(repository.full_name, issue_number,
                            comparison.title, comparison.content,
-                           assignee: comparison.assignee,
+                           assignee: comparison.assignees[0],
                            labels: comparison.labels)
+      if comparison.assignees.size > 1
+        @client.add_assignees(repository.full_name, issue_number,
+                              comparison.assignees[1..-1])
+      end
     end
 
     private

data/lib/sync_issues/issue.rb

@@ -6,27 +6,28 @@ module SyncIssues
   # new_title is only used when an issue should be renamed. Issues with
   # new_title set will never be created
   class Issue
-    attr_reader :assignee, :content, :labels, :new_title, :title
+    attr_reader :assignees, :content, :labels, :new_title, :title
 
-    def initialize(content, title:, assignee: nil, labels: nil, new_title: nil)
-      @assignee = verify_string 'assignee', assignee
+    def initialize(content, title:, assignees: nil, labels: nil, new_title: nil)
+      @assignees = verify_array_or_string 'assignees', assignees
+      @assignees.sort! unless @assignees.nil?
       @content = content
-      @labels = verify_labels(labels)
+      @labels = verify_array_or_string 'labels', labels
       @new_title = verify_string 'new_title', new_title, allow_nil: true
       @title = verify_string 'title', title, allow_nil: false
     end
 
     private
 
-    def verify_labels(labels)
-      return nil if labels.nil?
-      if labels.is_a?(String)
-        [verify_string('labels', labels, allow_nil: false)]
-      elsif !labels.is_a?(Array)
-        raise IssueError, "'labels' must be an Array or a String"
+    def verify_array_or_string(field, items)
+      return nil if items.nil?
+      if items.is_a?(String)
+        [verify_string(field, items, allow_nil: false)]
+      elsif !items.is_a?(Array)
+        raise IssueError, "'#{field}' must be an Array or a String"
       else
-        labels.each_with_index.map do |label, i|
-          verify_string("labels[#{i}]", label, allow_nil: false)
+        items.each_with_index.map do |item, i|
+          verify_string("#{field}[#{i}]", item, allow_nil: false)
         end
       end
     end

data/lib/sync_issues/synchronizer.rb

@@ -96,7 +96,7 @@ module SyncIssues
     def update_issue(repository, issue, github_issue)
       comparison = Comparison.new(issue, github_issue,
                                   reset_labels: @reset_labels,
-                                  sync_assignee: @sync_assignees,
+                                  sync_assignees: @sync_assignees,
                                   sync_labels: @sync_labels)
       return unless comparison.changed?
 

data/lib/sync_issues/version.rb

@@ -1,4 +1,4 @@
 # SyncIssues
 module SyncIssues
-  VERSION = '0.7.0'.freeze
+  VERSION = '0.8.0'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sync_issues
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Bryce Boe
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '4.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '4.8'
 - !ruby/object:Gem::Dependency
   name: safe_yaml
   requirement: !ruby/object:Gem::Requirement