changelogerator 0.0.3 → 0.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b916cdb658f760af9794c4f947c54839f8b19b596b3a0fe9e20f674c8898f31d
-  data.tar.gz: efa45a9492704d76197ccb4b3d15d227a6c4a01d35bfe8fe6ff81b7b84049532
+  metadata.gz: 6061881a5234a97b447b594f942e78a3bd4fcad68ed910265d12ed711eda8b64
+  data.tar.gz: f575aa00e5d4abbab3adaf529e309889f5261ced0e40da74186ab144563dcbe6
 SHA512:
-  metadata.gz: 298138ad4d9d52615de65931e3a96d7b5aee928bc71732a7142dfe2b8d97008d0dc090c2ef9ce81060b375288e3c5d9a4a3a375dafbca0b056d4b0b525ddcdc7
-  data.tar.gz: 70897f7bf61f7721a9d7097479e7c39c4be18c1781d970908b0620aadb2bf106d31ba81d0bc8e0e0eed988e38e0fda49a7f6549c38d511d0f8dd5e48db2fc93f
+  metadata.gz: d3f92505939fb6858ad0ce76f1e037973c07f7177034b3fc96c0189c55fcc4a109c34427306fffb7d8ebcb816551517065c0e3bc12651dc990aafacd0280b579
+  data.tar.gz: 6fae7c8942409d782ff606d67b3963a7869513529fd7f61caea8f7198f0c416b02196e0d08865b602e8d9c3c2ed8a35eb3d1cfe57359d147d7a147f318446efc

data/lib/changelogerator.rb

@@ -8,6 +8,7 @@
 # Probably not tremendously useful to other projects.
 class Changelog
   require 'octokit'
+  require 'git_diff_parser'
 
   attr_accessor :changes
   attr_reader :priority
@@ -51,10 +52,19 @@ class Changelog
 
   def self.changes_with_label(changes, label)
     changes.select do |change|
-      change.labels.any? { |c| c[:name] == label } == true
+      change[:labels].any? { |c| c[:name] == label } == true
     end
   end
 
+  def self.changes_files_in_paths?(change, paths)
+    changed_files = GitDiffParser.parse(Octokit.get(change.diff_url)).files
+    paths = [paths] unless paths.is_a? Array
+    paths.each do |path|
+      return true if changed_files.find { |l| l.match path }
+    end
+    nil
+  end
+
   ## End of class methods
 
   # github_repo: 'paritytech/polkadot'
@@ -70,7 +80,8 @@ class Changelog
     @gh = Octokit::Client.new(
       access_token: token
     )
-    @changes = prs_from_ids(pr_ids_from_git_diff(from, to), prefix)
+    @prefix = prefix
+    @changes = prs_from_ids(pr_ids_from_git_diff(from, to)).map(&:to_hash)
     # add priority to each change
     @changes.map { |c| apply_priority_to_change(c) }
   end
@@ -79,35 +90,61 @@ class Changelog
     self.class.changes_with_label(@changes, label)
   end
 
+  def runtime_changes?
+    nil
+  end
+
+  def add(change)
+    changes.prepend(apply_priority_to_change(change))
+  end
+
+  def add_from_id(id)
+    pull = @gh.pull_request(@repo, id)
+    add(prettify_title(pull))
+  end
+
   private
 
   def apply_priority_to_change(change)
     @priorities.each do |p|
-      change[:priority] = p if change.labels.any? { |l| l[:name] == p[:label] }
+      change[:priority] = p if change[:labels].any? { |l| l[:name] == p[:label] }
     end
+    # Failsafe: add lowest priority if none detected
+    change[:priority] ||= @priorities[0]
     change
   end
 
+  def prettify_title(pull)
+    pull[:pretty_title] = if @prefix
+                            "#{pull[:title]} (#{@repo}##{pull[:number]})"
+                          else
+                            "#{pull[:title]} (##{pull[:number]})"
+                          end
+    pull
+  end
+
   def pr_ids_from_git_diff(from, to)
     @gh.compare(@repo, from, to).commits.map do |c|
       title = c.commit.message.split("\n\n").first
       next unless title =~ /\(#[0-9]+\)$/
 
       title.gsub(/.*#([0-9]+)\)$/, '\1')
-    end.compact
+    end.compact.map(&:to_i)
   end
 
-  def prs_from_ids(ids, prefix)
+  def prs_from_ids(ids)
+    batch_size = 100
     prs = []
-    ids.each do |pr|
-      pull = @gh.pull_request(@repo, pr)
-      pull[:pretty_title] = if prefix
-                              "#{pull[:title]} (#{@repo}##{pull[:number]})"
-                            else
-                              "#{pull[:title]} (##{pull[:number]})"
-                            end
-      prs.push pull
+    @gh.pulls(@repo, state: 'closed', per_page: batch_size)
+    cur_batch = @gh.last_response
+    until ids.empty?
+      prs += cur_batch.data.select { |pr| ids.include? pr.number }
+      ids -= cur_batch.data.map(&:number)
+      break if cur_batch.rels[:last].nil?
+
+      cur_batch = cur_batch.rels[:next].get
     end
-    prs
+    prs.flatten
+    prs.map { |pr| prettify_title(pr) }
   end
 end

metadata

@@ -1,29 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: changelogerator
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.10
 platform: ruby
 authors:
 - Martin Pugh
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-09-02 00:00:00.000000000 Z
+date: 2020-10-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: github_api
+  name: git_diff_parser
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.19'
+        version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.19'
+        version: '3'
+- !ruby/object:Gem::Dependency
+  name: octokit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4'
 description: Simple helper class for paritytech/polkadot changelogs
 email:
 executables: []