git_ownership_insights 2.0.6 → 2.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9e95410a234ad6fdf3d9684a28da2dcd3515cb77c3f949fa0c2e97723cb6e76f
-  data.tar.gz: e212afe069ae204cd8f2eb8b07a49774dd64de590945940da812e6c4048ab111
+  metadata.gz: dbefa0c02b3e70f9172c29f63082848b17769bb3105245d1316d2ee9981c8a42
+  data.tar.gz: 289af8169edfe728fcf50e106816b99648a0dc545de8daadb4c52676302a54ab
 SHA512:
-  metadata.gz: 3b9ea8b6441df0167ff471a2118a293c5e250dbecc817a330985a29ab950d12b59d8de1c24cb2b2b275fbfc842500174deeb906d792e6d51338adc905c62d919
-  data.tar.gz: d08540070983653aa3216aab3a3e7b01d46e36e35666cc7ca0e7d4307e7d7995742628b781704397a8dcfa633cfce82cd8d34d2f78f33eb13bed559aaa7d810a
+  metadata.gz: 79f06802c24fd254a4962a039e2bf487be3e997606ccc1c46154542921921755fdc9d16735193629c0b61a724e2e2a82dcb483b925879e7ebf6523b0da41f206
+  data.tar.gz: 2982e6394672e9cf726a537895cda4b0ebccf63ddb570469651acfea1846540fcb1ac89c296d1ea85f4c879f1154dd3cd9d37aada9775889b85f6b6f4c65964d

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    git_ownership_insights (2.0.6)
+    git_ownership_insights (2.0.8)
       date
       pry
 

data/bin/git_ownership_insights

@@ -54,16 +54,6 @@ OptionParser.new do |opts|
     options[:team_regex] = team_regex
   end
 
-  opts.on('--top-contributing-team STRING',
-          'Limit of top contributed to the directory teams in codeownership data [default: 5]') do |top_contributing_team|
-    options[:top_contributing_team] = top_contributing_team
-  end
-
-  opts.on('--top-touched-files STRING',
-          'Limit of top touched files by individual contributors in codeownership data [default: 5]') do |top_touched_files|
-    options[:top_touched_files] = top_touched_files
-  end
-
   opts.on('--codeowners-path STRING', 'Path to CODEOWNERS file [default: .github/CODEOWNERS]') do |codeowners_path|
     options[:codeowners_path] = codeowners_path
   end
@@ -107,8 +97,6 @@ end.parse!
 EXCLUSIONS = options[:exclusions]&.split(',')
 REPO_PATH = options[:path] || '.'
 TEAM_REGEX = options[:team_regex] || '[A-Za-z]+'
-TOP_TOUCHED_FILES = options[:top_touched_files] || 5
-TOP_CONTRIBUTED_TEAMS = options[:top_contributing_team] || 5
 CODEOWNERS_PATH = options[:codeowners_path] || '.github/CODEOWNERS'
 BIG_FILE_SIZE = options[:big_file_size] || 250
 CI = options[:ci] || false
@@ -131,8 +119,6 @@ unless CI
   puts "Lines of code limit (big files) for the hotspot calculation: #{BIG_FILE_SIZE}"
   puts "Hotspot detailed output is: #{options[:hotspot_files] ? 'on' : 'off'}\n"
   puts "CODEOWNERS output is: #{options[:codeowners] ? 'on' : 'off'}\n"
-  puts "Limit of the teams shown in codeownership data: #{TOP_CONTRIBUTED_TEAMS}"
-  puts "Limit of the files shown in codeownership data: #{TOP_TOUCHED_FILES}"
   puts "Output to file mode is: #{options[:file_output] ? 'on' : 'off'}\n"
   puts "CI mode is: #{options[:ci] ? 'on' : 'off'}\n"
   puts "Debug mode is: #{options[:debug] ? 'on' : 'off'}\n\n"

data/lib/git_ownership_insights/git_ownership_insight.rb

@@ -75,13 +75,9 @@ class GitOwnershipInsights
 
     # Sort owners_data by total count in descending order
     sorted_owners_data = owners_data.sort_by { |_, data| -data[:churn_count] }
-
-    # Take the last 5 elements
-    top_owners_data = sorted_owners_data.last(TOP_CONTRIBUTED_TEAMS.to_i)
-
     converted_team_map = file_team_map.transform_keys { |key| File.basename(key) }
 
-    top_owners_data.each do |owner, data|
+    sorted_owners_data.each do |owner, data|
       output += "\n      #{owner.split('/').last}:\n         Total Count: #{data[:churn_count]}\n"
       data[:directories].each do |dir, dir_data|
         output += "            Directory: #{dir}\n               Top files:\n"
@@ -89,7 +85,7 @@ class GitOwnershipInsights
           next if converted_team_map[File.basename(file_data[:name])].nil?
 
           contributors = converted_team_map[file_data[:name]]&.first&.empty? ? ['Excluded contributor'] : converted_team_map[file_data[:name]].first
-          output += "                  #{File.basename(file_data[:name])} - #{file_data[:count].last} #{contributors}}"
+          output += "                  #{File.basename(file_data[:name])} - #{file_data[:count].last} #{contributors}}\n"
         end
       end
     end
@@ -268,7 +264,7 @@ class GitOwnershipInsights
         file = line.first
         contributors = line.last.first
         commits = line.last.last
-        hotspot_output += "     #{file.gsub(@directory_path, '')} Contributors: #{contributors} Commits: #{commits} Owner: #{find_owner(file:)}"
+        hotspot_output += "     #{file.gsub(@directory_path, '')} Contributors: #{contributors} Commits: #{commits} Owner: #{find_owner(file:)}\n"
       end
 
       if FILE_OUTPUT

data/lib/git_ownership_insights/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class GitOwnershipInsights
-  VERSION = '2.0.6'
+  VERSION = '2.0.8'
 end

data/spec/git_ownership_insights_spec.rb

@@ -5,8 +5,6 @@ require_relative '../lib/git_ownership_insights/git_ownership_insight'
 RSpec.describe GitOwnershipInsights do
   EXCLUSIONS = %w[exclusion1 exclusion2].freeze
   TEAM_REGEX = '[A-Za-z]+'
-  TOP_TOUCHED_FILES = 5
-  TOP_CONTRIBUTED_TEAMS = 5
   CODEOWNERS_PATH = '.github/CODEOWNERS'
   BIG_FILE_SIZE = 250
   CI = false
@@ -68,6 +66,7 @@ RSpec.describe GitOwnershipInsights do
                     Directory: spec/fixtures
                        Top files:
                           file1.swift - 3 ["FIOS", "FAND"]}
+
               FAND:
                  Total Count: 3
                     Directory: spec/fixtures

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: git_ownership_insights
 version: !ruby/object:Gem::Version
-  version: 2.0.6
+  version: 2.0.8
 platform: ruby
 authors:
 - Serghei Moret