git_ownership_insights 1.2.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6d884a43cd70caba8943ba59a6ce05d46631cb6927ed0eefa74a1dccbc2d66f8
-  data.tar.gz: 180007096b00a2978d8e6d6440c9214b1f8d59e41c841e665392db2c955fecd8
+  metadata.gz: 87339076047688d1452d5b7a1ca0455c9955cc580b9e472889d7dcf54f901fc7
+  data.tar.gz: cffcd3905dfd9dd500b92636c59301e7b7f640c594b3687f3770310d198eaa0b
 SHA512:
-  metadata.gz: 27a2bc913215ffb0cee443be41b1a91128edc31998d0f3841ad0ac02fd03393529f41825c55645cfc007f0a0d478a439834568334e2bf81c89929d3eeb396f37
-  data.tar.gz: e8d564d1734b023321027464ff86aea38463085681ef579297b69fc8f4bfd034b558d064e754bc0b142bf1a18171f4e959dd9c1af0d2a2ac34a375f01f5cbb88
+  metadata.gz: 71c488b0aff9354657f8a0b822490b9910b5082dfcbc9cf65ab521cb39f79f55a8441809101998166621b82a8dda5a0bc67c48adda796350fd8df6b5843ed0de
+  data.tar.gz: a41606244a1b77a1f62a08541cecadeec45f09f6eba2714d332f85359c6e670210b921e37854f64c445c0f42efd31def4d7f53a8804db989806f078f88bb5dd9

data/.github/CODEOWNERS

@@ -0,0 +1,2 @@
+spec/fixtures/file1.swift @FIOS
+spec/fixtures/file2.kt @FAND

data/.rubocop.yml

@@ -0,0 +1,14 @@
+inherit_from: .rubocop_todo.yml
+
+# The behavior of RuboCop can be controlled via the .rubocop.yml
+# configuration file. It makes it possible to enable/disable
+# certain cops (checks) and to alter their behavior if they accept
+# any parameters. The file can be placed either in your home
+# directory or in some project directory.
+#
+# RuboCop will start looking for the configuration file in the directory
+# where the inspected file is and continue its way up to the root directory.
+#
+# See https://docs.rubocop.org/rubocop/configuration
+AllCops:
+  TargetRubyVersion: 3.2.1

data/.rubocop_todo.yml

@@ -0,0 +1,65 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2024-01-30 15:39:52 UTC using RuboCop version 1.60.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 12
+# Configuration parameters: AllowedMethods.
+# AllowedMethods: enums
+Lint/ConstantDefinitionInBlock:
+  Exclude:
+    - 'spec/git_ownership_insights_spec.rb'
+
+# Offense count: 4
+# Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
+Metrics/AbcSize:
+  Max: 181
+
+# Offense count: 4
+# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
+# AllowedMethods: refine
+Metrics/BlockLength:
+  Max: 76
+
+# Offense count: 1
+# Configuration parameters: CountComments, CountAsOne.
+Metrics/ClassLength:
+  Max: 200
+
+# Offense count: 1
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/CyclomaticComplexity:
+  Max: 28
+
+# Offense count: 4
+# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
+Metrics/MethodLength:
+  Max: 109
+
+# Offense count: 1
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/PerceivedComplexity:
+  Max: 29
+
+# Offense count: 1
+# Configuration parameters: AllowedConstants.
+Style/Documentation:
+  Exclude:
+    - 'spec/**/*'
+    - 'test/**/*'
+    - 'lib/git_ownership_insights/git_ownership_insight.rb'
+
+# Offense count: 1
+Style/MultilineBlockChain:
+  Exclude:
+    - 'lib/git_ownership_insights/git_ownership_insight.rb'
+
+# Offense count: 11
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns.
+# URISchemes: http, https
+Layout/LineLength:
+  Max: 229

data/Gemfile.lock

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

data/bin/git_ownership_insights

@@ -134,5 +134,5 @@ end
 system("git checkout #{DEFAULT_BRANCH}", [:out] => File::NULL)
 system('git pull', %i[out err] => File::NULL)
 
-GitOwnershipInsights.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])
+GitOwnershipInsights.new(duration_in_days: options[:duration_in_days] || 30, directory_path: REPO_PATH,
+                         begin_time: DateTime.now, steps: options[:steps].to_i, debug: options[:debug]).contribution_message

data/lib/git_ownership_insights/git_ownership_insight.rb

@@ -1,11 +1,22 @@
 # frozen_string_literal: true
 
-module GitOwnershipInsights
-  def self.true?(obj)
+require 'pry'
+require 'date'
+
+class GitOwnershipInsights
+  def initialize(directory_path:, duration_in_days:, begin_time:, debug: nil, steps: 1)
+    @directory_path = directory_path
+    @duration_in_days = duration_in_days
+    @begin_time = begin_time
+    @debug = debug
+    @steps = steps
+  end
+
+  def true?(obj)
     obj.to_s.downcase == 'true'
   end
 
-  def self.read_codeowners_file
+  def read_codeowners_file
     raise "CODEOWNERS file does not exist under #{CODEOWNERS_PATH}" unless File.exist?(CODEOWNERS_PATH)
 
     codeowners = {}
@@ -20,7 +31,7 @@ module GitOwnershipInsights
     codeowners
   end
 
-  def self.find_owners(file_path, codeowners)
+  def find_owners(file_path, codeowners)
     matching_patterns = codeowners.keys.select do |pattern|
       pattern_regex = Regexp.new("^#{Regexp.escape(pattern.sub(%r{^/+}, '').chomp('/')).gsub('\*', '.*').gsub('**',
                                                                                                               '.*?')}")
@@ -42,7 +53,7 @@ module GitOwnershipInsights
     codeowners[best_match].split(' ')
   end
 
-  def self.count_big_files(directory_path, size: BIG_FILE_SIZE)
+  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) }
@@ -65,7 +76,7 @@ module GitOwnershipInsights
     puts "     *Current(*) total number of code files longer than #{size} lines:* #{count}"
   end
 
-  def self.count_hotspot_lines(files)
+  def count_hotspot_lines(files)
     code_files = files.select do |f|
       extension = File.extname(f)
       valid_extensions = CODE_EXTENSIONS
@@ -83,55 +94,57 @@ module GitOwnershipInsights
     puts "     *Total lines of hotspot code:* #{count}"
   end
 
-  def self.filter_existing_code_files(files)
+  def filter_existing_code_files(files)
     files.select do |f|
       next unless File.exist?(f)
 
+      if EXCLUDED_FILES
+        excluded_patterns = EXCLUDED_FILES.split(',')
+        next if excluded_patterns.any? { |pattern| f.include?(pattern) }
+      end
+
       extension = File.extname(f)
       valid_extensions = CODE_EXTENSIONS
       valid_extensions.include?(extension)
     end
   end
 
-  def self.git_files(directory_path:)
+  def git_files(directory_path:)
     `git ls-tree -r --name-only $(git rev-list -1 HEAD) -- "#{directory_path}"`
   end
 
-  def self.files_with_changes(directory_path:, start_date:, end_date:)
+  def files_with_changes(directory_path:, start_date:, end_date:)
     `git log --name-only --pretty=format:"" --since="#{start_date}" --until="#{end_date}" "#{directory_path}"`
   end
 
-  def self.git_commit_count(file:, start_date:, end_date:)
+  def git_commit_count(file:, start_date:, end_date:)
     `git log --since="#{start_date}" --until="#{end_date}" --follow -- "#{file}" | grep -c '^commit'`
   end
 
-  def self.contribution_message(directory_path:, duration_in_days:, begin_time:, debug: nil, steps: nil)
-    duration_in_days = duration_in_days.to_i
+  def git_commit_info(file:, start_date:, end_date:)
+    `git log --pretty=format:"%s" --since="#{start_date}" --until="#{end_date}" --follow -- "#{file}"`
+  end
+
+  def contribution_message
+    duration_in_days = @duration_in_days.to_i
     all_teams = []
     cross_teams_count = 0
     single_ownership_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 - 30 * 86_400
-    end_date = begin_time.to_time.to_i - 30 * 86_400
-    git_ls = git_files(directory_path:)
+    start_date = @begin_time.to_time.to_i - duration_in_days * 86_400 - 30 * 86_400
+    end_date = @begin_time.to_time.to_i - 30 * 86_400
+    git_ls = git_files(directory_path: @directory_path)
     file_count = filter_existing_code_files(git_ls.split).count
-    all_files_with_changes = files_with_changes(directory_path:, start_date:, end_date:).split.sort
+    all_files_with_changes = files_with_changes(directory_path: @directory_path, start_date:, end_date:).split.sort
     code_files_with_changes = filter_existing_code_files(all_files_with_changes)
     uniq_code_files_with_changes = code_files_with_changes.uniq
 
-    if EXCLUDED_FILES
-      excluded_patterns = EXCLUDED_FILES.split(',')
-      uniq_code_files_with_changes = uniq_code_files_with_changes.reject do |file|
-        excluded_patterns.any? { |pattern| file.include?(pattern) }
-      end
-    end
-
     file_team_map = {}
     uniq_code_files_with_changes.each do |file|
       filename = File.basename(file)
       commit_count = git_commit_count(file:, start_date:, end_date:).to_i
-      git_log = `git log --pretty=format:"%s" --since="#{start_date}" --until="#{end_date}" --follow -- "#{file}"`.split("\n")
+      git_log = git_commit_info(file:, start_date:, end_date:).split("\n")
       teams = git_log.map do |team|
         team.match(/#{TEAM_REGEX}/)[0].upcase
       end.reject { |e| EXCLUSIONS&.include?(e) }
@@ -148,7 +161,7 @@ module GitOwnershipInsights
         single_ownership_teams_count += 1
       end
 
-      puts "\n#{filename} [#{commit_count}]:#{teams}\n" if debug
+      puts "\n#{filename} [#{commit_count}]:#{teams}\n" if @debug
     end
 
     occurrences = all_teams.flatten.compact.tally
@@ -169,7 +182,7 @@ module GitOwnershipInsights
     filtered_top_touched_files = filtered_files.sort_by { |element, count| [-count.last, element] }
 
     puts ''
-    puts "*Timeframe:* #{(begin_time - duration_in_days).strftime('%Y-%m-%d')} to #{begin_time.strftime('%Y-%m-%d')}"
+    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 / file_count) * 100)).round(2)}%"
     puts "        *Existing files changed by many teams:* #{files_changed_by_many_teams}"
     puts "        *Current existing #{CODE_EXTENSIONS} files:* #{file_count}"
@@ -184,7 +197,7 @@ module GitOwnershipInsights
     puts "     *#{CODE_EXTENSIONS} files exceeding #{BIG_FILE_SIZE} lines with multiple contributors:* #{filtered_top_touched_files.count}"
     puts "     *Total amount of commits to #{CODE_EXTENSIONS} files:* #{total_changes}"
     puts "     *Total #{CODE_EXTENSIONS} files changed:* #{uniq_code_files_with_changes.count}"
-    count_big_files(directory_path)
+    count_big_files(@directory_path)
     puts "     *Current(*) total of #{CODE_EXTENSIONS} files in the folder:* #{file_count}"
     puts "     *Contributors:* #{contributors}"
     puts "* means that it the current(instant) repository value, all the other metrics are done over #{duration_in_days} days period"
@@ -193,7 +206,7 @@ module GitOwnershipInsights
       puts "\n"
       puts '   Hotspot changes:'
       filtered_top_touched_files.each do |line|
-        puts "     #{line.first.gsub(directory_path, '')} Contributors: #{line.last.first} Commits: #{line.last.last}"
+        puts "     #{line.first.gsub(@directory_path, '')} Contributors: #{line.last.first} Commits: #{line.last.last}"
       end
     end
 
@@ -240,13 +253,13 @@ module GitOwnershipInsights
         end
       end
     end
-    steps -= 1
+    @steps -= 1
 
-    return unless steps.positive?
+    return unless @steps.positive?
 
-    system("git checkout `git rev-list -1 --before='#{(begin_time - duration_in_days).strftime('%B %d %Y')}' HEAD`",
+    system("git checkout `git rev-list -1 --before='#{(@begin_time - duration_in_days).strftime('%B %d %Y')}' HEAD`",
            %i[out err] => File::NULL)
-    contribution_message(duration_in_days:, directory_path:,
-                         begin_time: begin_time - duration_in_days, steps:, debug:)
+    @begin_time -= duration_in_days
+    contribution_message
   end
 end

data/lib/git_ownership_insights/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
-module GitOwnershipInsights
-  VERSION = '1.2.0'
+class GitOwnershipInsights
+  VERSION = '2.0.1'
 end

data/lib/git_ownership_insights.rb

@@ -2,7 +2,7 @@
 
 require_relative 'git_ownership_insights/version'
 
-module GitOwnershipInsights
+class GitOwnershipInsights
   class Error < StandardError; end
   # Your code goes here...
 end

data/sig/git_ownership_insights.rbs

@@ -1,4 +1,4 @@
-module GitOwnershipInsights
+class GitOwnershipInsights
   VERSION: String
   # See the writing guide of rbs: https://github.com/ruby/rbs#guides
 end

data/spec/fixtures/file1.swift

@@ -0,0 +1,333 @@
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+line
+// comment
+
+
+line
+line
+line
+line
+line
+line
+line
+line
+line