github_repo_statistics 2.0.8 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 259f604375dde894d7103813f052aeb59b5317afc190856ed3cbea8eccc2d1d3
-  data.tar.gz: 1b5ae5061a41f3cc8985691c25cce3615bd93b477be8c2c1917fe0565dd8b4d7
+  metadata.gz: 7625d04f395885b90b8f2faad896127df5e16604818d095f3e5455bf49d9c899
+  data.tar.gz: e6fec8b96326f6aff20e8ba63ccde41d3605381a692df51a3e7dceac8f0b218c
 SHA512:
-  metadata.gz: 66bdf997d8853d016513c603b5264d6b30816d99c419c73d9c2ab2a6ebe7e6a36f1e25c9d2faf58ad12b4e16c2da83dabe2ede7183404bb8f880555655d4b920
-  data.tar.gz: 9c6b4d5d36eeeb509294542b36bd9ce636b8d0adbfd4808d3a87d7de05fb0b1001a763ddc234116e9879a70c4c8e4370f666bb3d696e013d2b36b2784de741a1
+  metadata.gz: c943a3541d786ca121518c4545a0d82f036d71492ad83bec46ffffd187e4f6536475688acd1f3841e20ded27b29eb4759675e40ef86d55a66ef2ae147487e8fe
+  data.tar.gz: 76c00e2a9cc37a21026cb31229f6c7c71733428b1fc1d06fd24f5c8e9d5c8777d85d23e45c4af73ddaf699d6c8afd63dab38dc40af2780c0f73f704e32b0b4b2

data/.gitignore

@@ -0,0 +1,2 @@
+.idea/*
+*.gem

data/.rubocop.yml

@@ -0,0 +1 @@
+inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -0,0 +1,67 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2024-02-27 08:25:07 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: 11
+# Configuration parameters: AllowedMethods.
+# AllowedMethods: enums
+Lint/ConstantDefinitionInBlock:
+  Exclude:
+    - 'spec/github_repo_statistics.rb'
+
+# Offense count: 8
+# Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
+Metrics/AbcSize:
+  Max: 96
+
+# Offense count: 7
+# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
+# AllowedMethods: refine
+Metrics/BlockLength:
+  Max: 73
+
+# Offense count: 1
+# Configuration parameters: CountComments, CountAsOne.
+Metrics/ClassLength:
+  Max: 229
+
+# Offense count: 4
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/CyclomaticComplexity:
+  Max: 18
+
+# Offense count: 8
+# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
+Metrics/MethodLength:
+  Max: 62
+
+# Offense count: 4
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/PerceivedComplexity:
+  Max: 18
+
+# Offense count: 3
+# Configuration parameters: AllowedConstants.
+Style/Documentation:
+  Exclude:
+    - 'spec/**/*'
+    - 'test/**/*'
+    - 'lib/github_repo_statistics/force_merge_report.rb'
+    - 'lib/github_repo_statistics/github_repo_statistics.rb'
+    - 'lib/github_repo_statistics/review_report.rb'
+
+# Offense count: 1
+Style/MultilineBlockChain:
+  Exclude:
+    - 'lib/github_repo_statistics/github_repo_statistics.rb'
+
+# Offense count: 13
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns.
+# URISchemes: http, https
+Layout/LineLength:
+  Max: 229

data/Gemfile.lock

@@ -1,20 +1,33 @@
 PATH
   remote: .
   specs:
-    github_repo_statistics (2.0.8)
+    github_repo_statistics (2.1.0)
       date
+      faraday-retry
+      octokit
       pry
 
 GEM
   remote: https://rubygems.org/
   specs:
+    addressable (2.8.4)
+      public_suffix (>= 2.0.2, < 6.0)
     ast (2.4.2)
     coderay (1.1.3)
     date (3.3.4)
     diff-lcs (1.5.0)
+    faraday (2.7.9)
+      faraday-net_http (>= 2.0, < 3.1)
+      ruby2_keywords (>= 0.0.4)
+    faraday-net_http (3.0.2)
+    faraday-retry (2.2.0)
+      faraday (~> 2.0)
     json (2.7.1)
     language_server-protocol (3.17.0.3)
     method_source (1.0.0)
+    octokit (8.0.0)
+      faraday (>= 1, < 3)
+      sawyer (~> 0.9)
     parallel (1.24.0)
     parser (3.3.0.4)
       ast (~> 2.4.1)
@@ -22,6 +35,7 @@ GEM
     pry (0.14.2)
       coderay (~> 1.1)
       method_source (~> 1.0)
+    public_suffix (5.0.1)
     racc (1.7.3)
     rainbow (3.1.1)
     rake (13.1.0)
@@ -54,6 +68,10 @@ GEM
     rubocop-ast (1.30.0)
       parser (>= 3.2.1.0)
     ruby-progressbar (1.13.0)
+    ruby2_keywords (0.0.5)
+    sawyer (0.9.2)
+      addressable (>= 2.3.5)
+      faraday (>= 0.17.3, < 3)
     unicode-display_width (2.5.0)
 
 PLATFORMS

data/README.md

@@ -1,39 +1,107 @@
 # GithubRepoStatistics
 
-TODO: Delete this and the text below, and describe your gem
-
-Welcome to your new gem! In this directory, you'll find the files you need to be able to package up your Ruby library into a gem. Put your Ruby code in the file `lib/github_repo_statistics`. To experiment with that code, run `bin/console` for an interactive prompt.
+This is a standalone ruby executable that allows you to collect statistics 
 
 ## Installation
+*Prerequisite:* You will need at least Ruby 3.1.0 to run this gem. You can install it throug `rbenv` or `rvm`. (e.g. `brew install rvm && rvm install 3.2.2 && rvm use defaukt 3.2.2`)
 
-TODO: Replace `UPDATE_WITH_YOUR_GEM_NAME_PRIOR_TO_RELEASE_TO_RUBYGEMS_ORG` with your gem name right after releasing it to RubyGems.org. Please do not do it earlier due to security reasons. Alternatively, replace this section with instructions to install your gem from git if you don't plan to release to RubyGems.org.
-
-Install the gem and add to the application's Gemfile by executing:
-
-    $ bundle add UPDATE_WITH_YOUR_GEM_NAME_PRIOR_TO_RELEASE_TO_RUBYGEMS_ORG
-
-If bundler is not being used to manage dependencies, install the gem by executing:
-
-    $ gem install UPDATE_WITH_YOUR_GEM_NAME_PRIOR_TO_RELEASE_TO_RUBYGEMS_ORG
+*Installing the gem:*
+`gem install github_repo_statistics`
 
 ## Usage
 
-TODO: Write usage instructions here
-
-## Development
-
-After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
-
-To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and the created tag, and push the `.gem` file to [rubygems.org](https://rubygems.org).
-
-## Contributing
-
-Bug reports and pull requests are welcome on GitHub at https://github.com/[USERNAME]/github_repo_statistics. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/[USERNAME]/github_repo_statistics/blob/main/CODE_OF_CONDUCT.md).
-
-## License
-
-The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).
-
-## Code of Conduct
-
-Everyone interacting in the GitOwnershipInsights project's codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/[USERNAME]/github_repo_statistics/blob/main/CODE_OF_CONDUCT.md).
+```
+github_repo_statistics -h
+Usage: github_repo_statistics [options]
+        --debug                      Enable debug mode
+        --ci                         Do not print the info messages for better CI text parsing [default: false]
+        --codeowners                 Print CODEOWNERS info [default: false]
+        --hotspot-files              Print the found hotspot files (big files touched by many) [default: false]
+        --excluded-contributors STRING
+                                     Comma-delimited list of excluded contributors [example: WEB,RAILS,MOBILE]
+        --excluded-files STRING      Comma-delimited list of excluded keywords [example: ViewController,AppDelegate.swift]
+        --steps STRING               Number of steps the script will go into the past [default: 1]
+        --duration-in-days STRING    Number of days to aggregate the changes for [default: 30]
+        --path STRING                Path to the directory or file to calculate the ownership [default: "."]
+        --team-regex STRING          Regex that will identify the team name [default: "[A-Za-z]+"]
+        --codeowners-path STRING     Path to CODEOWNERS file [default: .github/CODEOWNERS]
+        --big-file-size STRING       The amount of lines in the file to be considered big [default: 250]
+        --default-branch STRING      The default branch to pull and run metrics for [default: master]
+        --code-extensions STRING     The file extensions that consider to be code [default: ".kt,.swift"]
+        --output-to-files            Puts the output for hotspot and codeowners into files instead of the STDOUT (useful for CI and big amount of data) [default:false]
+    -v, --version                    Display the version of the gem
+    -h, --help                       Display this help message
+
+Examples of usage:
+  github_repo_statistics --path src/test --exclusions WEB,RAILS --steps 2 --duration-in-days 90 --hotspot-files --debug
+
+
+*Timeframe:* 2023-11-01 to 2024-01-30
+     *Code files with a single contributor:* 94.4%
+        *Existing files changed by many teams:* 101
+        *Current existing [".swift"] files:* 1805
+     *Cross-Squad Dependency:*
+        *Contributions by multiple squads to the same files:* 282
+        *Contributions by single squads contributing to single files:* 501
+     *Hotspot Code Changes:* 42.48%
+        *Churn count(commits to files by multiple teams):* 701
+        *Total amount of commits:* 1650
+     *Total lines of hotspot code:* 19965
+     *[".swift"] files with multiple contributors:* 101
+     *[".swift"] files exceeding 250 lines with multiple contributors:* 37
+     *Total amount of commits to [".swift"] files:* 1650
+     *Total [".swift"] files changed:* 602
+     *Current total number of code files longer than 250 lines:* 102
+     *Current total of [".swift"] files in the folder:* 1805
+     *Contributors:* {"CCE"=>245, "SQDREACT"=>145, "DSAT"=>124, "PROD"=>111, "ORGANIC"=>92, "CS"=>75, "SF"=>61, "PCON"=>55, "ER"=>48, "VIRRCH"=>45, "PACC"=>37, "MN"=>31, "CAWLAPP"=>28, "CMT"=>25, "PAYS"=>24, "EX"=>12, "REF"=>11, "ZEST"=>3, "UFI"=>2, "VRIRRCH"=>2, "CHANGED"=>1, "CLEANUPS"=>1, "DPLAN"=>1}
+```
+
+```
+force_merge_report --help
+Usage: github_repo_statistics [options]
+        --ci                         Do not print the info messages for better CI text parsing [default: false]
+        --default-branch STRING      The default branch to pull and run metrics for [default: master]
+        --github-token STRING        GitHub API token [default: ""]
+        --duration-in-days STRING    Number of days to aggregate the changes for [default: 90]
+        --github-repo STRING         GitHub repository name and owner (e.g. octokit/octokit) [default: ""]
+    -v, --version                    Display the version of the gem
+    -h, --help                       Display this help message
+    
+Examples of usage:
+    force_merge_report --github-token ghp_SAKLGSJHG8asgiasKASGkhsa --github-repo octokit/octokit --duration-in-days 7
+    
+PR #12315 - Merged at: 2024-02-26 16:20:22 UTC
+Failed Checks:
+- Workflow: Mergeable: Size check, Conclusion: failure
+---
+
+PR #312455 - Merged at: 2024-02-26 15:01:09 UTC
+Failed Checks:
+- Workflow: Mergeable: PR Review, Conclusion: failure
+---
+
+Weekly Summary:
+Calendar week 2022-05: Total PRs: 11, Force-merged PRs: 2
+Failed Workflows:
+  - Mergeable: Size check: 1
+  - Mergeable: PR Review: 1
+```
+
+```
+review_report -h
+Usage: github_repo_statistics [options]
+        --duration-in-days STRING    Number of days to aggregate the changes for [default: 90]
+        --default-branch STRING      The default branch to pull and run metrics for [default: master]
+        --github-token STRING        GitHub API token [default: ""]
+        --github-repo STRING         GitHub repository name and owner (e.g. octokit/octokit) [default: ""]
+    -v, --version                    Display the version of the gem
+    -h, --help                       Display this help message
+
+Examples of usage:
+    review_report --github-token ghp_SAKLGSJHG8asgiasKASGkhsa --github-repo octokit/octokit
+                         
+Calendar week 2023-04: Average time taken for 24 pull requests is 2.49 hours.
+  Total reviews: 440
+  Reviews with comments: 247
+  Change requested reviews: 7
+```

data/bin/force_merge_report

@@ -0,0 +1,61 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+require 'date'
+require 'optparse'
+require 'pry'
+require_relative '../lib/github_repo_statistics/version'
+require_relative '../lib/github_repo_statistics/force_merge_report'
+
+options = {}
+OptionParser.new do |opts|
+  opts.banner = 'Usage: github_repo_statistics [options]'
+
+  opts.on('--ci', 'Do not print the info messages for better CI text parsing [default: false]') do
+    options[:ci] = true
+  end
+
+  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('--github-token STRING',
+          'GitHub API token [default: ""]') do |github_token|
+    options[:github_token] = github_token
+  end
+
+  opts.on('--duration-in-days STRING',
+          'Number of days to aggregate the changes for [default: 90]') do |duration_in_days|
+    options[:duration_in_days] = duration_in_days.to_i
+  end
+
+  opts.on('--github-repo STRING',
+          'GitHub repository name and owner (e.g. octokit/octokit) [default: ""]') do |github_repo|
+    options[:github_repo] = github_repo
+  end
+
+  opts.on('-v', '--version', 'Display the version of the gem') do
+    puts "github_repo_statistics version #{GithubRepoStatistics::VERSION}"
+    exit
+  end
+
+  opts.on('-h', '--help', 'Display this help message') do
+    puts opts
+    puts <<~EXAMPLES
+
+      Examples:
+        force_merge_report --github-token ghp_bKasj29ndkfdksf3rjt2ngi43j --github-repo octokit/octokit
+    EXAMPLES
+    exit
+  end
+end.parse!
+
+DEFAULT_BRANCH = options[:default_branch] || 'master'
+
+raise 'Please provide GitHub token using --github-token flag' if options[:github_token].nil?
+raise 'Please provide GitHub repo name using --github-repo flag' if options[:github_repo].nil?
+raise 'Please provide default GitHub branch using --default-branch flag' if DEFAULT_BRANCH.nil?
+
+ForceMergeReport.new(token: options[:github_token], repo: options[:github_repo],
+                     branch: DEFAULT_BRANCH, duration: options[:duration_in_days] || 30, ci: options[:ci]).report

data/bin/review_report

@@ -0,0 +1,58 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+require 'date'
+require 'optparse'
+require 'pry'
+require_relative '../lib/github_repo_statistics/version'
+require_relative '../lib/github_repo_statistics/force_merge_report'
+require_relative '../lib/github_repo_statistics/review_report'
+
+options = {}
+OptionParser.new do |opts|
+  opts.banner = 'Usage: github_repo_statistics [options]'
+
+  opts.on('--duration-in-days STRING',
+          'Number of days to aggregate the changes for [default: 90]') do |duration_in_days|
+    options[:duration_in_days] = duration_in_days.to_i
+  end
+
+  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('--github-token STRING',
+          'GitHub API token [default: ""]') do |github_token|
+    options[:github_token] = github_token
+  end
+
+  opts.on('--github-repo STRING',
+          'GitHub repository name and owner (e.g. octokit/octokit) [default: ""]') do |github_repo|
+    options[:github_repo] = github_repo
+  end
+
+  opts.on('-v', '--version', 'Display the version of the gem') do
+    puts "github_repo_statistics version #{GithubRepoStatistics::VERSION}"
+    exit
+  end
+
+  opts.on('-h', '--help', 'Display this help message') do
+    puts opts
+    puts <<~EXAMPLES
+
+      Examples:
+        review_report --github-token ghp_bKasj29ndkfdksf3rjt2ngi43j --github-repo octokit/octokit
+    EXAMPLES
+    exit
+  end
+end.parse!
+
+DEFAULT_BRANCH = options[:default_branch] || 'master'
+
+raise 'Please provide GitHub token using --github-token flag' if options[:github_token].nil?
+raise 'Please provide GitHub repo name using --github-repo flag' if options[:github_repo].nil?
+raise 'Please provide default GitHub branch using --default-branch flag' if DEFAULT_BRANCH.nil?
+
+ReviewReport.new(token: options[:github_token], repo: options[:github_repo], branch: DEFAULT_BRANCH,
+                 duration: options[:duration_in_days] || 30).report

data/github_repo_statistics.gemspec

@@ -25,5 +25,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'date'
+  spec.add_dependency 'octokit'
   spec.add_dependency 'pry'
+  spec.add_dependency 'faraday-retry'
 end

data/lib/github_repo_statistics/force_merge_report.rb

@@ -0,0 +1,87 @@
+# frozen_string_literal: true
+
+require 'octokit'
+require 'date'
+
+class ForceMergeReport
+  def initialize(token:, repo:, branch:, duration:, ci:)
+    @token = token
+    @repo = repo
+    @branch = branch
+    @duration_in_days = duration
+    @ci = ci
+  end
+
+  def report
+    client = Octokit::Client.new(access_token: @token)
+    client.auto_paginate = true
+
+    # Get all pull requests created after the target date
+    pull_requests = client.list_issues(@repo, state: 'closed', since: DateTime.now - @duration_in_days)
+
+    # Sort PRs into monthly chunks
+    weekly_pull_requests = pull_requests.group_by { |pr| pr.closed_at.strftime('%Y-%U') }
+
+    # Initialize a hash to store monthly summaries
+    weekly_summaries = Hash.new { |hash, key| hash[key] = { total: 0, failed: 0, workflows: Hash.new(0) } }
+
+    weeks = @duration_in_days / 7
+
+    (weekly_pull_requests.keys.sort[-weeks..] || []).each do |month|
+      pull_requests_for_month = weekly_pull_requests[month]
+      # Iterate through pull requests for the current month
+      pull_requests_for_month.each do |issue|
+        # Get the pull request details
+        pr = client.pull_request(@repo, issue.number)
+
+        # Check if the pull request is merged
+        next unless pr.merged_at
+
+        # Get the check runs for the pull request's head commit
+        check_runs = client.check_runs_for_ref(@repo, pr.head.sha)
+        latest_check_runs = check_runs.check_runs.uniq(&:name)
+
+        statuses = client.statuses(@repo, pr.head.sha)
+        latest_statuses = statuses.uniq(&:context)
+
+        all_checks = latest_check_runs + latest_statuses
+
+        # Filter checks without meeting the required status checks
+        failed_checks = all_checks.reject { |check| check.conclusion == 'success' || check.state == 'success' }
+
+        # Update monthly summary
+        weekly_summaries[month][:total] += 1
+        weekly_summaries[month][:failed] += 1 unless failed_checks.empty?
+
+        # Update workflow counts
+        failed_checks.each do |check|
+          workflow_name = check.name || check.context
+          weekly_summaries[month][:workflows][workflow_name] += 1
+        end
+
+        # Print details of merged pull requests without meeting the required criteria for the last 6 months
+        next if failed_checks.empty?
+
+        unless @ci
+          puts "PR ##{pr.number} - Merged at: #{pr.merged_at}"
+          puts 'Failed Checks:'
+          failed_checks.each do |check|
+            puts "- Workflow: #{check.name || check.context}, Conclusion: #{check.conclusion || check.state}"
+          end
+          puts '---'
+          puts
+        end
+      end
+    end
+
+    # Print the weekly summary
+    puts 'Weekly Summary:'
+    weekly_summaries.each do |month, summary|
+      puts "Calendar week #{month}: Total PRs: #{summary[:total]}, Force-merged PRs: #{summary[:failed]}"
+      puts 'Failed Workflows:'
+      summary[:workflows].each do |workflow, count|
+        puts "  - #{workflow}: #{count}"
+      end
+    end
+  end
+end

data/lib/github_repo_statistics/github_repo_statistics.rb

@@ -205,7 +205,8 @@ class GithubRepoStatistics
 
       puts "\n#{filename} [#{commit_count}]:#{teams}\n" if @debug
     end
-    [all_teams, cross_teams_count, single_ownership_teams_count, files_changed_by_many_teams, total_changes, file_team_map]
+    [all_teams, cross_teams_count, single_ownership_teams_count, files_changed_by_many_teams, total_changes,
+     file_team_map]
   end
 
   def filter_files(file_team_map:)
@@ -226,7 +227,9 @@ class GithubRepoStatistics
     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
-    all_teams, cross_teams_count, single_ownership_teams_count, files_changed_by_many_teams, total_changes, file_team_map = analyze_changed_files(uniq_code_files_with_changes:, start_date:, end_date:)
+    all_teams, cross_teams_count, single_ownership_teams_count, files_changed_by_many_teams, total_changes, file_team_map = analyze_changed_files(
+      uniq_code_files_with_changes:, start_date:, end_date:
+    )
     occurrences = all_teams.flatten.compact.tally
     sorted_occurrences = occurrences.sort_by { |element, count| [-count, element] }
     contributors = Hash[sorted_occurrences]
@@ -264,7 +267,8 @@ class GithubRepoStatistics
         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:)}\n"
+        hotspot_output += "     #{file.gsub(@directory_path,
+                                            '')} Contributors: #{contributors} Commits: #{commits} Owner: #{find_owner(file:)}\n"
       end
 
       if FILE_OUTPUT

data/lib/github_repo_statistics/review_report.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+require 'octokit'
+require 'date'
+
+class ReviewReport
+  def initialize(token:, repo:, branch:, duration:)
+    @token = token
+    @repo = repo
+    @branch = branch
+    @duration_in_days = duration
+  end
+
+  def percentile(array, percentile)
+    sorted_array = array.sort
+    k = (percentile * (sorted_array.length - 1)).floor
+    l = k + 1
+    q1 = sorted_array[k]
+    q2 = sorted_array[l]
+    q1 + (q2 - q1) * (percentile * (sorted_array.length - 1) - k)
+  end
+
+  def report
+    client = Octokit::Client.new(access_token: @token)
+    client.auto_paginate = true
+
+    # Fetch pull requests created after the target date
+    pull_requests = client.list_issues(@repo, state: 'closed', since: DateTime.now - @duration_in_days)
+
+    # Group pull requests by week
+    pull_requests_by_week = pull_requests.group_by { |pr| pr[:closed_at].strftime('%Y-%U') }
+
+    weeks = @duration_in_days / 7
+
+    # Iterate over months within the days duration limit
+    pull_requests_by_week.keys.sort[-weeks..].each do |week|
+      total_time_seconds = 0
+      total_count = 0
+      total_count_calc = 0
+      total_reviews = 0
+      reviews_with_comments = 0
+      change_requested_reviews = 0
+
+      # Iterate over pull requests in the week
+      pull_requests_by_week[week].each do |issue|
+        pull_request = client.pull_request(@repo, issue.number)
+        last_commit_sha = pull_request[:head][:sha]
+        last_commit = client.commit(@repo, last_commit_sha)
+
+        reviews = []
+        review_page = 1
+        loop do
+          response = client.pull_request_reviews(@repo, pull_request[:number], per_page: 100,
+                                                                               page: review_page)
+          break if response.empty?
+
+          reviews.concat(response)
+          review_page += 1
+        end
+
+        total_reviews += reviews.size
+
+        next unless reviews.any?
+
+        last_review = reviews.max_by { |review| review[:submitted_at] }
+        time_taken_seconds = (last_review[:submitted_at] - last_commit[:commit][:author][:date]).abs
+
+        if time_taken_seconds / 3600.0 < 8 && time_taken_seconds / 3600.0 > 0.3
+          total_time_seconds += time_taken_seconds
+          total_count_calc += 1
+        end
+        total_count += 1
+        # Count reviews with comments
+        reviews_with_comments += reviews.count do |review|
+          comments = client.pull_request_review_comments(@repo, pull_request[:number], review[:id])
+          comments.any?
+        end
+
+        # Count reviews with "change_requested" state
+        change_requested_reviews += reviews.count { |review| review[:state] == 'CHANGES_REQUESTED' }
+      end
+
+      if total_count.positive?
+        average_time_hours = total_time_seconds / total_count_calc / 3600.0 # Convert seconds to hours
+        puts "Calendar week #{week}: Average time taken for #{total_count} pull requests is #{average_time_hours.round(2)} hours."
+        puts "  Total reviews: #{total_reviews}"
+        puts "  Reviews with comments: #{reviews_with_comments}"
+        puts "  Change requested reviews: #{change_requested_reviews}"
+      else
+        puts "Month #{week}: No pull requests with reviews."
+      end
+    end
+  end
+end

data/lib/github_repo_statistics/version.rb

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

data/spec/github_repo_statistics.rb

@@ -5,7 +5,7 @@ require_relative '../lib/github_repo_statistics/github_repo_statistics'
 RSpec.describe GithubRepoStatistics do
   EXCLUSIONS = %w[exclusion1 exclusion2].freeze
   TEAM_REGEX = '[A-Za-z]+'
-  CODEOWNERS_PATH = '.github/CODEOWNERS'
+  CODEOWNERS_PATH = 'spec/fixtures/CODEOWNERS'
   BIG_FILE_SIZE = 250
   CI = false
   DEFAULT_BRANCH = 'main'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: github_repo_statistics
 version: !ruby/object:Gem::Version
-  version: 2.0.8
+  version: 2.1.0
 platform: ruby
 authors:
 - Serghei Moret
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-23 00:00:00.000000000 Z
+date: 2024-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: date
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: octokit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -38,20 +52,38 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: faraday-retry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description:
 email:
 - serghei.moret@hellofresh.com
 executables:
 - console
+- force_merge_report
 - github_repo_statistics
+- review_report
 - setup
 extensions: []
 extra_rdoc_files: []
 files:
-- ".github/CODEOWNERS"
 - ".github/workflows/hf_orc-gating.yaml"
 - ".github/workflows/hf_validate-repository-ownership.yaml"
 - ".github/workflows/hf_validate-tribe-squad-labels.yaml"
+- ".gitignore"
+- ".rubocop.yml"
+- ".rubocop_todo.yml"
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Gemfile
@@ -60,13 +92,18 @@ files:
 - README.md
 - Rakefile
 - bin/console
+- bin/force_merge_report
 - bin/github_repo_statistics
+- bin/review_report
 - bin/setup
 - github_repo_statistics.gemspec
 - lib/github_repo_statistics.rb
+- lib/github_repo_statistics/force_merge_report.rb
 - lib/github_repo_statistics/github_repo_statistics.rb
+- lib/github_repo_statistics/review_report.rb
 - lib/github_repo_statistics/version.rb
 - sig/github_repo_statistics.rbs
+- spec/fixtures/CODEOWNERS
 - spec/fixtures/file.wrong
 - spec/fixtures/file1.swift
 - spec/fixtures/file2.kt
@@ -102,6 +139,7 @@ signing_key:
 specification_version: 4
 summary: This gem prints git ownership insights
 test_files:
+- spec/fixtures/CODEOWNERS
 - spec/fixtures/file.wrong
 - spec/fixtures/file1.swift
 - spec/fixtures/file2.kt