changelogerator 0.0.16 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7139ba15a0c62ce2c9df1eb5a59312b68572c0a92ed0dbbb9190689b58281e18
-  data.tar.gz: 4feefa1c8e1d0b8104c21e566af4aee6fb10d898109945453a9927d96f20a7cf
+  metadata.gz: b292e9657833099e9623ee12bc856d1212fe047a0fc3da90ef7192ab305c2f6d
+  data.tar.gz: 24fab3a02631ae80b4ab8b810b8aee5b9fd9340280bfce389247dfb19d215ef6
 SHA512:
-  metadata.gz: da1d4ac8c642a14b81a986af888cd852d476073968311bd4e0cdf9685c983529ac9dbfa27cafefbd5c1cd9e1a3af7c97d54b82c9a7d8c6280845a4f9022b7e5b
-  data.tar.gz: ba9f28d314db9dcb7a01763ce921da9b01a9ee1d9cfa40ed0ee04d160bb467b4e1966824bcb3211f83ca8060111d84a440cf972db2a8de4ea3253a1fba5084d3
+  metadata.gz: 28564cd68a1ebd4d4917e8f43339551a144edac68fa242ce1b3248e0f123e9835040d8c267e51755bfb0a3e09a26fb22214605ace98212b9bdc573b54c0265a0
+  data.tar.gz: b9c23322c65362bdee0f6535982d4c349bde39092f99457901bebe48d5ba9d77e5484faab552ed81a5f786976024dc68f600eeaa85a34a578a50df6ec560f7dd

data/bin/changelogerator

@@ -0,0 +1,73 @@
+#!/usr/bin/env ruby
+
+# frozen_string_literal: true
+
+require 'optparse'
+require_relative '../lib/changelogerator'
+require 'logger'
+
+@options = {
+  from: nil,
+  to: nil
+}
+
+logger = Logger.new($stdout)
+logger.level = Logger::WARN
+logger.debug('Starting')
+
+OptionParser.new do |opts|
+  opts.banner = "
+  Changelogerator helps you generate a template friendly context made of the changes
+  between 2 references on your Github project.
+
+  Usage:
+  changelogerator <repo> [options]"
+
+  opts.on('-v', '--[no-]verbose', 'Run verbosely') do |v|
+    @options[:verbose] = v
+  end
+
+  opts.on('-fFROM', '--from=FROM', 'From Ref.') do |f|
+    @options[:from] = f
+  end
+
+  opts.on('-tTO', '--to=TO', 'To Ref.') do |t|
+    @options[:to] = t
+  end
+
+  opts.on('-p', '--[no-]prefix', 'Add (or not) a prefix for the repo') do |p|
+    @options[:prefix] = p
+  end
+
+  opts.on('-h', '--help', 'Prints this help') do
+    puts opts
+    exit
+  end
+end.parse!
+
+@options[:repo] = ARGV[0]
+
+def validate
+  raise "Missing value for the 'repo' field" unless @options[:repo]
+  raise "Missing value for the 'from' field" unless @options[:from]
+  raise "Missing value for the 'to' field" unless @options[:to]
+
+  true
+end
+
+exit 1 unless validate
+
+logger.debug('Validation passed')
+logger.debug(format(' - repo: %s', @options[:repo]))
+logger.debug(format(' - from: %s', @options[:from]))
+logger.debug(format(' - to: %s', @options[:to]))
+
+cl = Changelog.new(
+  @options[:repo],
+  @options[:from],
+  @options[:to],
+  token: ENV['GITHUB_TOKEN'],
+  prefix: @options[:prefix]
+)
+
+puts cl.to_json

data/lib/changelogerator.rb

@@ -1,54 +1,19 @@
 # frozen_string_literal: true
 
+require 'pry'
+
 # A small wrapper class for more easily generating and manipulating Github/Git
 # changelogs. Given two different git objects (sha, tag, whatever), it will
 # find all PRs that made up that diff and store them as a list. Also allows
-# for filtering by label, and the importance of that change (priorities), based
+# for filtering by label, and the importance of that change (labels), based
 # on how we classify the importance of PRs in the paritytech/polkadot project.
-# Probably not tremendously useful to other projects.
 class Changelog
   require 'octokit'
   require 'git_diff_parser'
+  require 'json'
 
-  attr_accessor :changes
-  attr_reader :priority
-
-  @priorities = [
-    {
-      priority: 1,
-      label: 'C1-low 📌',
-      text: 'Upgrade priority: **Low** (upgrade at your convenience)'
-    },
-    {
-      priority: 3,
-      label: 'C3-medium 📣',
-      text: 'Upgrade priority: **Medium** (timely upgrade recommended)'
-    },
-    {
-      priority: 7,
-      label: 'C7-high ❗️',
-      text: 'Upgrade priority:❗ **HIGH** ❗ Please upgrade your node as soon as possible'
-    },
-    {
-      priority: 9,
-      label: 'C9-critical ‼️',
-      text: 'Upgrade priority: ❗❗ **URGENT** ❗❗ PLEASE UPGRADE IMMEDIATELY'
-    }
-  ]
-
-  class << self
-    attr_reader :priorities
-  end
-
-  # Return highest priority from an array of changes (NOT the actual Changelog
-  # object)
-  def self.highest_priority_for_changes(changes)
-    @priorities.find do |p|
-      p[:priority] == changes.map do |change|
-        change[:priority][:priority]
-      end.max
-    end || @priorities[0]
-  end
+  attr_accessor :repository, :changes, :meta
+  attr_reader :label
 
   def self.changes_with_label(changes, label)
     changes.select do |change|
@@ -56,6 +21,32 @@ class Changelog
     end
   end
 
+  # Go through all changes and compute some
+  # aggregated values
+  def compute_global_meta
+    @meta = {}
+
+    @changes.each do |change|
+      # here we remove some of the fields to reduce (considerably) the size
+      # of the json output
+      change.head = nil
+      change.base = nil
+      change._links = nil
+
+      change[:meta].each_key do |meta_key|
+        current = change[:meta][meta_key]
+
+        meta[meta_key] = {} unless meta[meta_key]
+        meta[meta_key][:min] = current[:value] if !meta[meta_key][:min] || current[:value] < meta[meta_key][:min]
+        meta[meta_key][:max] = current[:value] if !meta[meta_key][:max] || current[:value] > meta[meta_key][:max]
+        meta[meta_key][:count] = 0 unless meta[meta_key][:count]
+        meta[meta_key][:count] += 1
+      end
+    end
+  end
+
+  # Return the list of all the files in the changeset
+  # that also in the given path
   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
@@ -65,6 +56,18 @@ class Changelog
     nil
   end
 
+  # Return the label code for a change
+  # if the label name matches the expected pattern.
+  # nil otherwise.
+  def self.get_label_code(name)
+    m = match = name.match(/^([a-z])(\d+)-(.*)$/i)
+    if m
+      letter, number, text = match.captures
+      return [letter, number, text]
+    end
+    nil
+  end
+
   ## End of class methods
 
   # github_repo: 'paritytech/polkadot'
@@ -76,44 +79,71 @@ class Changelog
   # prefix: whether or not to prefix PR numbers with their repo in the changelog
   def initialize(github_repo, from, to, token: '', prefix: nil)
     @repo = github_repo
-    @priorities = self.class.priorities
     @gh = Octokit::Client.new(
       access_token: token
     )
+    @repository = @gh.repository(@repo)
     @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
-
-  def changes_with_label(label)
-    self.class.changes_with_label(@changes, label)
-  end
+    @changes.map do |c|
+      compute_change_meta(c)
+    end
 
-  def runtime_changes?
-    nil
+    compute_global_meta
   end
 
   def add(change)
-    changes.prepend(prettify_title(apply_priority_to_change(change)))
+    compute_change_meta(change)
+    prettify_title(change)
+    changes.prepend(change)
+    @meta = compute_global_meta
   end
 
+  # Add a pull request from id
   def add_from_id(id)
     pull = @gh.pull_request(@repo, id)
     add pull
   end
 
+  def to_json(*_args)
+    opts = {
+      array_nl: "\n",
+      object_nl: "\n",
+      indent: '  ',
+      space_before: ' ',
+      space: ' '
+    }
+    obj = @changes
+
+    commits = {
+      meta: @meta,
+      repository: @repository.to_h,
+      changes: obj.map(&:to_h)
+    }
+
+    JSON.fast_generate(commits, opts)
+  end
+
   private
 
-  def apply_priority_to_change(change)
-    @priorities.each do |p|
-      change[:priority] = p if change[:labels].any? { |l| l[:name] == p[:label] }
+  # Compute and attach metadata about one change
+  def compute_change_meta(change)
+    meta = {}
+
+    change.labels.each do |label|
+      letter, number, text = self.class.get_label_code(label.name)
+      next unless letter && number
+
+      meta[letter] = {
+        value: number.to_i,
+        text: text
+      }
     end
-    # Failsafe: add lowest priority if none detected
-    change[:priority] ||= @priorities[0]
-    change
+
+    change['meta'] = meta
   end
 
+  # Prepend the repo if @prefix is true
   def prettify_title(pull)
     pull[:pretty_title] = if @prefix
                             "#{pull[:title]} (#{@repo}##{pull[:number]})"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: changelogerator
 version: !ruby/object:Gem::Version
-  version: 0.0.16
+  version: 0.9.0
 platform: ruby
 authors:
 - Martin Pugh
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-25 00:00:00.000000000 Z
+date: 2021-11-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: git_diff_parser
@@ -40,10 +40,12 @@ dependencies:
         version: '4'
 description: Simple helper class for paritytech/polkadot changelogs
 email:
-executables: []
+executables:
+- changelogerator
 extensions: []
 extra_rdoc_files: []
 files:
+- bin/changelogerator
 - lib/changelogerator.rb
 homepage: https://github.com/s3krit/changelogerator
 licenses:
@@ -57,14 +59,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.2.22
 signing_key:
 specification_version: 4
 summary: Changelog generation/management