changelogerator 0.0.1 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 168c71f208452944f40001c41d2902d879c19b2f01e9a0a523d9d9a5b37def22
-  data.tar.gz: bd5d94e6d95e19eab2cb6b115ad286c6f8dd7e343f72646b79f6936e454978ef
+  metadata.gz: 400a2661d78f9c51ff0a2c4dacc852d464a9997f29c8017df1f5d585318504ff
+  data.tar.gz: 1c32b784170d2eb50a1edea3aaee8e2b3d30321c5539209606913ea788ece2a3
 SHA512:
-  metadata.gz: 6b2ab20548258e78d78ac99deed1bc509f5bbf8bbbd062b141a92f86990d6e7d8261d108bc4a51091b9998808aefbed7388eb21b4da0045d3ecea430b7508865
-  data.tar.gz: 360a16cd39c8e224a3c0e66d6ad1d9b1d454cc1605b576acc91dd30c941ea2a23adbe4c653f70e3b02a58d73b4ba8d83891f30d5f5db20e6de2d74b6b95bc520
+  metadata.gz: 538053876b3075cf839b80812a108854ff54a172683bd4c155cc7f4e17d52b6036746817742c0b956744a91861af870466a9873bba11db11866bacfb2086810f
+  data.tar.gz: 79dd9b05c967834aa77b76ef08a83f0eccaaae33364743eb1512c2992ef49ab128dacae9c3303c85e647ed5126a62818ed66d633338eaedb889d5a424f51675c

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
@@ -55,6 +56,15 @@ class Changelog
     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))
     # add priority to each change
     @changes.map { |c| apply_priority_to_change(c) }
   end
@@ -82,6 +90,19 @@ 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)
@@ -91,26 +112,37 @@ class Changelog
     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,15 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: changelogerator
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.8
 platform: ruby
 authors:
 - Martin Pugh
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-07-28 00:00:00.000000000 Z
-dependencies: []
+date: 2020-10-01 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: git_diff_parser
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        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: []