changelogerator 0.0.2 → 0.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: abb70fa6b03f1b32056fe0c98ca923521c1a1f34799ce26d620a691386645b99
-  data.tar.gz: f1f9879fee9e9e105ef470d7f1c62240f9bf1270918cdfd0c5dbabbe4f8c6b28
+  metadata.gz: 5d7e604fe48a69c44b84032971ca11723132e4570f558fb501205947f31e5b26
+  data.tar.gz: fd9f8601e21a0d927e00cb4887656664b40f2d7b22ef48af9e93bbcada39d29e
 SHA512:
-  metadata.gz: 4986ec720e213717191e03077ce3aae3bea1e43b50d0411fdc0d7c4e3fedaf47f10cb0ec89a4372c1540f185b7ad3d3c7acdd216aaacbc456bbb5698d1a22396
-  data.tar.gz: c462ba5d117b9f37659407473b78f1a91b2a8aba54dc89ff2b7d33d3de0a9aa6d32d606fd1d7001dcb328906f7c19d8b7aaf30e159e3c3ed21a5c55abbb8f489
+  metadata.gz: dcf89c2dbf2dc1ad2d43664a97b54a802f65b92056643f2ae6fb4ae5c5224470783faf26b268acc2d3db748691d241ca7ac46a8c35c6d69933645a33c8765a41
+  data.tar.gz: d9d256e73a61f8de59d077f390715522ea4a944f97f595b24a30be713aca77278a3c298f86dbfe5a6dd7bd01f99a4291fbe78b46551dfa83a87eec604d77f773

data/lib/changelogerator.rb

@@ -7,7 +7,8 @@
 # on how we classify the importance of PRs in the paritytech/polkadot project.
 # Probably not tremendously useful to other projects.
 class Changelog
-  require 'github_api'
+  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'
@@ -65,15 +75,13 @@ class Changelog
   # token: a Github personal access token
   # prefix: whether or not to prefix PR numbers with their repo in the changelog
   def initialize(github_repo, from, to, token: '', prefix: nil)
-    org, repo = github_repo.split('/')
-    @ghr = github_repo
+    @repo = github_repo
     @priorities = self.class.priorities
-    @gh = Github.new do |c|
-      c.oauth_token = token
-      c.org = org
-      c.repo = repo
-    end
-    @changes = prs_from_ids(pr_ids_from_git_diff(from, to), prefix)
+    @gh = Octokit::Client.new(
+      access_token: token
+    )
+    @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
@@ -82,35 +90,59 @@ 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
     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.repos.commits.compare(@gh.org, @gh.repo, from, to).body.commits.map do |l|
-      title = l.commit.message.split("\n\n").first
+    @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_requests.get(@gh.org, @gh.repo, pr).body
-      pull[:pretty_title] = if prefix
-                              "#{pull[:title]} (#{@ghr}##{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.2
+  version: 0.0.9
 platform: ruby
 authors:
 - Martin Pugh
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-07-28 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: []