git_ownership_insights 1.0.3 → 1.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c8e4215542d1b1f05793def9ec7da4bf9c9cbdcc8b8fd0e00a912188ce877b85
-  data.tar.gz: 0b10b78de68c0f71013a9d6a9458b45dd824ed283bd3b0613c93f3146b0d1fc8
+  metadata.gz: dc9c0c038149e42cd56b515168124dd68d50476169deedd38e98cb8593770106
+  data.tar.gz: bc3e24fb12aac9bf7e94e6649b61b4941d1dfba20a7cc23f4ba56e566631faa1
 SHA512:
-  metadata.gz: f0cd127fa863e8d06d868527b5d3f2ea662f560d6149f95bbef1df18ed3442c7d08b6eb27aa4ed8e223c5bdbb2772a37aa1c8de41fb6920e5f96d7fd3555e833
-  data.tar.gz: 58fcc55d06921ce4c57c008496c82af548992d2e007b24d3449a2b51b1957ac41aa5a0a87bf809d7435814cf5273e8e70aff80659bc5786772e4c8a144687ae3
+  metadata.gz: 7638cb2568573d64238ba56438b9bb70e7a7ec6c9d28ac353e79c867d174039c04237103fc424cadd146d169bbce2014a8651445cc660ae2b7a06a7126bdb7cd
+  data.tar.gz: 5941142ba797b288b7a82e5a5b20edf91eb2c00cf754375743eccbb70e0ad31b8ce111f03f1c238746724d096f686b3f0b786c75432d3571e9b290eea7ae1284

data/Gemfile.lock

@@ -1,8 +1,7 @@
 PATH
   remote: .
   specs:
-    git_ownership_insights (0.1.4)
-      awesome_print
+    git_ownership_insights (1.0.4)
       date
       pry
 
@@ -10,7 +9,6 @@ GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.2)
-    awesome_print (1.9.2)
     coderay (1.1.3)
     date (3.3.4)
     diff-lcs (1.5.0)

data/bin/git_ownership_insights

@@ -10,11 +10,11 @@ options = {}
 OptionParser.new do |opts|
   opts.banner = 'Usage: git_ownership_insights [options]'
 
-  opts.on('-d', '--debug', 'Enable debug mode') do
+  opts.on('--debug', 'Enable debug mode') do
     options[:debug] = true
   end
 
-  opts.on('-ci', '--ci', 'Do not print the info messages for better CI text parsing [default: false]') do
+  opts.on('--ci', 'Do not print the info messages for better CI text parsing [default: false]') do
     options[:ci] = true
   end
 
@@ -22,56 +22,56 @@ OptionParser.new do |opts|
     options[:codeowners] = true
   end
 
-  opts.on('-hf', '--hotspot-files', 'Print the found hotspot files (big files touched by many) [default: false]') do
+  opts.on('--hotspot-files', 'Print the found hotspot files (big files touched by many) [default: false]') do
     options[:hotspot_files] = true
   end
 
-  opts.on('-ec', '--excluded-contributors STRING', 'Comma-delimited list of excluded contributors [example: WEB,RAILS,MOBILE]') do |exclusions|
+  opts.on('--excluded-contributors STRING', 'Comma-delimited list of excluded contributors [example: WEB,RAILS,MOBILE]') do |exclusions|
     options[:exclusions] = exclusions
   end
 
-  opts.on('-ef', '--excluded-files STRING', 'Comma-delimited list of excluded files [example: ViewController,AppDelegate.swift]') do |excluded_files|
+  opts.on('--excluded-files STRING', 'Comma-delimited list of excluded files [example: ViewController,AppDelegate.swift]') do |excluded_files|
     options[:excluded_files] = excluded_files
   end
 
-  opts.on('-s', '--steps STRING', 'Number of steps the script will go into the past [default: 1]') do |steps|
+  opts.on('--steps STRING', 'Number of steps the script will go into the past [default: 1]') do |steps|
     options[:steps] = steps
   end
 
-  opts.on('-d', '--duration-in-days STRING',
+  opts.on('--duration-in-days STRING',
           'Number of days to aggregate the changes for [default: 30]') do |duration_in_days|
     options[:duration_in_days] = duration_in_days
   end
 
-  opts.on('-p', '--path STRING', 'Path to the directory or file to calculate the ownership [default: "."]') do |path|
+  opts.on('--path STRING', 'Path to the directory or file to calculate the ownership [default: "."]') do |path|
     options[:path] = path
   end
 
-  opts.on('-reg', '--team-regex STRING', 'Regex that will identify the team name [default: "[A-Za-z]+"]') do |team_regex|
+  opts.on('--team-regex STRING', 'Regex that will identify the team name [default: "[A-Za-z]+"]') do |team_regex|
     options[:team_regex] = team_regex
   end
 
-  opts.on('-t', '--top-contributing-team STRING', 'Limit of top contributed to the directory teams in codeownership data [default: 5]') do |top_contributing_team|
+  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('-files', '--top-touched-files STRING', 'Limit of top touched files by individual contributors in codeownership data [default: 5]') do |top_touched_files|
+  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('-cp', '--codeowners-path STRING', 'Path to CODEOWNERS file [default: .github/CODEOWNERS]') do |codeowners_path|
+  opts.on('--codeowners-path STRING', 'Path to CODEOWNERS file [default: .github/CODEOWNERS]') do |codeowners_path|
     options[:codeowners_path] = codeowners_path
   end
 
-  opts.on('-s', '--big-file-size STRING', 'The amount of lines in the file to be considered big [default: 250]') do |big_file_size|
+  opts.on('--big-file-size STRING', 'The amount of lines in the file to be considered big [default: 250]') do |big_file_size|
     options[:big_file_size] = big_file_size
   end
 
-  opts.on('-b', '--default-branch STRING', 'The default branch to pull and run metrics for [default: master]') do |default_branch|
+  opts.on('--default-branch STRING', 'The default branch to pull and run metrics for [default: master]') do |default_branch|
     options[:default_branch] = default_branch
   end
 
-  opts.on('-ext', '--code-extensions STRING', 'The file extensions that consider to be code [default: ".kt, .swift"]') do |code_extension|
+  opts.on('--code-extensions STRING', 'The file extensions that consider to be code [default: ".kt, .swift"]') do |code_extension|
     options[:code_extension] = code_extension
   end
 
@@ -145,6 +145,7 @@ def find_owners(file_path, codeowners)
 end
 
 def count_big_files(directory_path, size: BIG_FILE_SIZE)
+  size = size.to_i
   # Get a list of all files in the specified directory
   files = Dir.glob(File.join(directory_path, '**', '*')).select { |file| File.file?(file) }
 
@@ -166,12 +167,33 @@ def count_big_files(directory_path, size: BIG_FILE_SIZE)
     end
   end
 
-  puts "   Total number of files longer than #{size} lines: #{count}"
+  puts "   Total number of code files longer than #{size} lines: #{count}"
+end
+
+def count_hotspot_lines(files)
+  code_files = files.select {|f|
+    extension = File.extname(f)
+    valid_extensions = ['.swift', '.kt']
+    valid_extensions.include?(extension)
+  }
+
+  # Initialize a counter for files that meet the criteria
+  count = 0
+
+  # Iterate through each file and check the line count
+  code_files.each do |file|
+    lines_count = File.foreach(file).reject { |line| line.match(/^\s*(\/\/|\/\*.*\*\/|\s*$)/) }.count
+
+    count += lines_count
+  end
+
+  puts "   Total lines of hotspot code: #{count}"
 end
 
 def contribution_message(directory_path:, duration_in_days:, begin_time:, debug: nil, steps: nil)
   duration_in_days = duration_in_days.to_i
   all_teams = []
+  teams_count = 0
   files_changed_by_many_teams = 0
   total_changes = 0
   start_date = begin_time.to_time.to_i - duration_in_days * 86_400
@@ -212,6 +234,7 @@ def contribution_message(directory_path:, duration_in_days:, begin_time:, debug:
     if teams.count > 1
       files_changed_by_many_teams += 1
       file_team_map.merge!("#{file}" => [teams, commit_count])
+      teams_count += teams.count
     end
 
     puts "\n#{filename} [#{commit_count}]:#{teams}\n" if debug
@@ -224,7 +247,14 @@ def contribution_message(directory_path:, duration_in_days:, begin_time:, debug:
   churn_count = file_team_map.values.map { |value| value[1] }.sum
   hotspot_changes_percentage = (churn_count.to_f / total_changes.to_f)*100
 
-  puts "Timeframe: #{(begin_time - duration_in_days).strftime('%Y-%m-%d')} to #{begin_time.strftime('%Y-%m-%d')}\n   Code files with a single contributor: #{(100 - ((files_changed_by_many_teams.to_f / code_files_with_changes.count.to_f) * 100)).round(2)}%\n   Hotspot code changes: #{churn_count} (#{hotspot_changes_percentage.round(2)}%)\n   Amount of code changes: #{total_changes}\n   Total files changed: #{code_files_with_changes.count}\n   Total files in the folder: #{file_count}\n   Contributors: #{contributors}\n"
+  puts "Timeframe: #{(begin_time - duration_in_days).strftime('%Y-%m-%d')} to #{begin_time.strftime('%Y-%m-%d')}"
+  puts "   Code files with a single contributor: #{(100 - ((files_changed_by_many_teams.to_f / code_files_with_changes.count.to_f) * 100)).round(2)}%"
+  puts "   Hotspot code changes: #{churn_count} (#{hotspot_changes_percentage.round(2)}%)"
+  puts "   Total occurrences of cross-squad dependencies over same files: #{teams_count}"
+  puts "   Amount of code changes: #{total_changes}"
+  puts "   Total files changed: #{code_files_with_changes.count}"
+  puts "   Total files in the folder: #{file_count}"
+  puts "   Contributors: #{contributors}"
 
   # Filter files based on extension and size
   filtered_files = file_team_map.select do |file_path|
@@ -236,6 +266,7 @@ def contribution_message(directory_path:, duration_in_days:, begin_time:, debug:
 
   filtered_top_touched_files = filtered_files.sort_by { |element, count| [-count.last, element] }
   count_big_files(directory_path)
+  count_hotspot_lines(filtered_files.keys)
   puts "   Total files longer than #{BIG_FILE_SIZE} lines with multiple contributors: #{filtered_top_touched_files.count}\n"
   if HOTSPOT
     filtered_top_touched_files.each do |line|
@@ -313,7 +344,7 @@ unless CI
 end
 
 system("git checkout #{DEFAULT_BRANCH}", [ :out ] => File::NULL)
-system("git pull", [ :out ] => File::NULL)
+system("git pull", [ :out, :err ] => File::NULL)
 
 contribution_message(duration_in_days: options[:duration_in_days] || 30, directory_path: REPO_PATH,
                      begin_time: DateTime.now, steps: options[:steps].to_i, debug: options[:debug])

data/lib/git_ownership_insights/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module GitOwnershipInsights
-  VERSION = '1.0.3'
+  VERSION = '1.0.5'
 end

metadata

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