cobench 0.0.17 → 0.0.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b84694d76fb95dc4c62d1c4ba42d4d4bee5c8c193c33c241b02964d44816e91
-  data.tar.gz: a56793bef0c5d3bdda384ddd553a47cf135c51cead18e73ed984a7a5f4922077
+  metadata.gz: 944037515588754a9074051087759954ea921346dbf852b234a9507836e4ad4d
+  data.tar.gz: e718f120d970ae6f60cd83c748d4f33b6570fc4befa9ec7801316700951b56bb
 SHA512:
-  metadata.gz: 4fee03b92b8a370f39bc207012bf777b54f459c903b08e25e75e469d98f458d599c85f2bccb51892c165b837192a3450da23d545f84c33263c07bf7469ed6261
-  data.tar.gz: 71bb57a42b734c46f865beda63815ad1984964bf96a690a4c94c9e67ca6eb32199a59c7efb12e0c5fa99423cd54f7087aa72c2443864ea96ad091fb2e7124bc3
+  metadata.gz: 9bed4c1f78e45b2d7185e0c2bb67b928bb4b642cc70ac49851ed81de577817ce54da6fdd2f85f836fb26c30d27814b68c4d6e6164907e69157be0cd209227751
+  data.tar.gz: 9768224ff1925aef1a9cfa7656c930ae3782e6f5488aa7aa502f5a8032974925da8f64916147609cc0a2f801afd466d1b6b707ba1a6343a9a063fdf49ff9aed8

data/.rultor.yml

@@ -1,3 +1,5 @@
+docker:
+  image: yegor256/rultor-image:1.9.1
 assets:
   rubygems.yml: yegor256/home#assets/rubygems.yml
 install: |
@@ -16,7 +18,3 @@ release:
 merge:
   script: |-
     bundle exec rake
-deploy:
-  script: |-
-    echo "There is nothing to deploy"
-    exit -1

data/README.md

@@ -26,6 +26,8 @@ Then, run it locally and read its output:
 $ cobench --coder yegor256 --verbose
 ```
 
+This is how our report [looks like](https://github.com/cqfn/bench).
+
 ## How to contribute
 
 Read [these guidelines](https://www.yegor256.com/2014/04/15/github-guidelines.html).

data/assets/index.xsl

@@ -82,11 +82,11 @@ SOFTWARE.
               <xsl:text> on </xsl:text>
               <xsl:value-of select="cobench/@time"/>
               <xsl:text>. </xsl:text>
-              <xsl:text>"Commits" is the total number of </xsl:text>
+              <xsl:text>"Commits" is the total number of non-merge </xsl:text>
               <a href="https://github.com/git-guides/git-commit">
                 <xsl:text>Git commits</xsl:text>
               </a>
-              <xsl:text> authored by the user. </xsl:text>
+              <xsl:text> to the default branch, authored by the user. </xsl:text>
               <xsl:text>"HoC" is the total number of user's </xsl:text>
               <a href="https://www.yegor256.com/2014/11/14/hits-of-code.html">
                 <xsl:text>hits of code</xsl:text>
@@ -104,11 +104,34 @@ SOFTWARE.
               <xsl:text> created by the user and already merged. </xsl:text>
               <xsl:text>"Reviews" is the total number of merged pull requests that were reviewed by the user. </xsl:text>
               <xsl:text>"Score" is an arithmetic summary of all metrics with multipliers: </xsl:text>
-              <xsl:text>one Pull costs 100 points, </xsl:text>
-              <xsl:text>one Issue — 50 points, </xsl:text>
-              <xsl:text>one Review — 40 points, </xsl:text>
-              <xsl:text>one Commit — 5 points, </xsl:text>
-              <xsl:text>one HoC — just 1 point.</xsl:text>
+              <xsl:for-each select="cobench/weights/w">
+                <xsl:if test="position() &gt; 1">
+                  <xsl:text>, </xsl:text>
+                </xsl:if>
+                <xsl:text>one </xsl:text>
+                <xsl:choose>
+                  <xsl:when test="substring(@id, string-length(@id)) = 's'">
+                    <xsl:value-of select="substring(@id, 0, string-length(@id))"/>
+                  </xsl:when>
+                  <xsl:otherwise>
+                    <xsl:value-of select="@id"/>
+                  </xsl:otherwise>
+                </xsl:choose>
+                <xsl:choose>
+                  <xsl:when test="position() = 1">
+                    <xsl:text> costs </xsl:text>
+                  </xsl:when>
+                  <xsl:otherwise>
+                    <xsl:text> — </xsl:text>
+                  </xsl:otherwise>
+                </xsl:choose>
+                <xsl:value-of select="text()"/>
+                <xsl:text> point</xsl:text>
+                <xsl:if test="text() != '1'">
+                  <xsl:text>s</xsl:text>
+                </xsl:if>
+              </xsl:for-each>
+              <xsl:text>.</xsl:text>
             </p>
             <p>
               <xsl:text>The numbers you see reflect the activity of the last </xsl:text>
@@ -164,7 +187,7 @@ SOFTWARE.
     </tr>
   </xsl:template>
   <xsl:template match="m">
-    <td class="num">
+    <xsl:variable name="body">
       <xsl:choose>
         <xsl:when test="@href = ''">
           <xsl:value-of select="."/>
@@ -175,6 +198,24 @@ SOFTWARE.
           </a>
         </xsl:otherwise>
       </xsl:choose>
+    </xsl:variable>
+    <td class="num">
+      <xsl:if test="@actual">
+        <xsl:choose>
+          <xsl:when test="@actual">
+            <span class="firebrick">
+              <xsl:value-of select="$body"/>
+            </span>
+          </xsl:when>
+          <xsl:otherwise>
+            <xsl:value-of select="$body"/>
+          </xsl:otherwise>
+        </xsl:choose>
+        <br/>
+        <span class="subtitle" title="The actual value of the metric was capped">
+          <xsl:value-of select="@actual"/>
+        </span>
+      </xsl:if>
     </td>
   </xsl:template>
   <xsl:template match="node()|@*">

data/bin/cobench

@@ -35,7 +35,7 @@ loog = Loog::REGULAR
 def config(path)
   f = File.expand_path(path)
   args = []
-  args += File.readlines(f).map(&:strip) if File.exist?(f)
+  args += File.readlines(f).map(&:strip).reject { |a| a.empty? } if File.exist?(f)
   args
 end
 
@@ -47,6 +47,7 @@ opts = Slop.parse(args, strict: true, help: true) do |o|
   o.bool '--version', 'Show current version'
   o.bool '--verbose', 'Print as much log messages as possible'
   o.bool '--dry', 'Make no real round trips to GitHub'
+  o.bool '--reuse', 'Don\'t fetch from GitHub, reuse the existing XML file'
   o.integer '--days', 'How many days to measure', default: 7
   o.string '--to', 'Directory where to save all files to', default: './cobench'
   o.string '--token', 'GitHub authentication token'
@@ -73,17 +74,7 @@ end
 Encoding.default_external = Encoding::UTF_8
 Encoding.default_internal = Encoding::UTF_8
 
-data = {}
-
-begin
-  home = File.absolute_path(opts[:to])
-  loog.debug("All files generated will be saved to #{home}")
-  if File.exist?(home)
-    loog.debug("Directory #{home} exists")
-  else
-    FileUtils.mkdir_p(home)
-    loog.debug("Directory #{home} created")
-  end
+def build_xml(opts, loog)
   if opts.token?
     api = Octokit::Client.new(:access_token => opts[:token])
   else
@@ -94,6 +85,7 @@ begin
   api = Obk.new(api, pause: 2000)
   loog.info("Reading GitHub data for the last #{opts[:days]} days")
   titles = {}
+  data = {}
   opts[:coder].each do |u|
     user = u.downcase
     loog.info("Scanning #{user}...")
@@ -111,9 +103,10 @@ begin
       if opts.dry?
         measures = [
           { title: 'Issues', total: Random.new.rand(100), href: 'https://github.com/' },
-          { title: 'Pulls', total: Random.new.rand(100), href: '' },
-          { title: 'HoC', total: Random.new.rand(100), href: '' },
-          { title: 'HoC', total: Random.new.rand(100), href: '' }
+          { title: 'Pulls', total: Random.new.rand(100) },
+          { title: 'Commits', total: Random.new.rand(100) },
+          { title: 'HoC', total: Random.new.rand(100) },
+          { title: 'HoC', total: Random.new.rand(100) }
         ]
       else
         measures = m.take(loog)
@@ -127,21 +120,31 @@ begin
       end
     end
   end
+  caps = {
+    'HoC' => lambda { |ms| ms['Commits'][:total] * 512 },
+  }
   data.each do |u, ms|
-    score = ms.map do |t, h|
-      h[:total] * if t == 'HoC'
-        1
-      elsif t == 'Pulls'
-        100
-      elsif t == 'Issues'
-        50
-      elsif t == 'Commits'
-        5
-      elsif t == 'Reviews'
-        40
-      else
-        raise "Unknown title '#{t}'"
+    ms.map do |t, h|
+      next unless caps.key?(t)
+      cap = caps[t].call(ms)
+      if h[:total] > cap
+        data[u][t][:actual] = h[:total]
+        data[u][t][:total] = cap
       end
+    end
+    data[u] = ms
+  end
+  weights = {
+    'HoC' => 1,
+    'Pulls' => 200,
+    'Issues' => 50,
+    'Commits' => 5,
+    'Reviews' => 75
+  }
+  data.each do |u, ms|
+    score = ms.map do |t, h|
+      raise "Unknown title '#{t}'" unless weights.key?(t)
+      h[:total] * weights[t]
     end.inject(0, :+)
     data[u]['Score'] = { total: score, href: '' }
   end
@@ -152,12 +155,22 @@ begin
           xml.title t
         end
       end
+      xml.weights do
+        weights.each do |t, w|
+          xml.w(id: t) do
+            xml.text w
+          end
+        end
+      end
       xml.coders do
         data.each do |u, ms|
-          xml.coder(id: u, details: api.user(u).name) do
+          xml.coder(id: u) do
+            xml.parent.set_attribute('details', api.user(u).name)
             xml.metrics do
               ms.each do |k, v|
                 xml.m(id: k, href: v[:href]) do
+                  xml.parent.set_attribute('actual', v[:actual]) unless v[:actual].nil?
+                  xml.parent.set_attribute('href', v[:href]) unless v[:href].nil?
                   xml.text v[:total]
                 end
               end
@@ -167,11 +180,27 @@ begin
       end
     end
   end
-  index = File.join(home, 'index.xml')
   xml = builder.to_xml
   loog.debug(xml)
-  File.write(index, xml)
-  loog.debug("XML saved to #{index} (#{File.size(index)} bytes)")
+  xml
+end
+begin
+  home = File.absolute_path(opts[:to])
+  loog.debug("All files generated will be saved to #{home}")
+  if File.exist?(home)
+    loog.debug("Directory #{home} exists")
+  else
+    FileUtils.mkdir_p(home)
+    loog.debug("Directory #{home} created")
+  end
+  index = File.join(home, 'index.xml')
+  if opts[:reuse]
+    xml = File.read(index)
+  else
+    xml = build_xml(opts, loog)
+    File.write(index, xml)
+    loog.debug("XML saved to #{index} (#{File.size(index)} bytes)")
+  end
   xslt = Nokogiri::XSLT(File.read(File.join(__dir__, '../assets/index.xsl')))
   html = xslt.transform(Nokogiri::XML(xml), 'version' => "'#{Cobench::VERSION}'")
   loog.debug(html)

data/features/cli.feature

@@ -11,7 +11,7 @@ Feature: Simple Reporting
     Then Exit code is zero
 
   Scenario: Simple report
-    When I run bin/cobench with "--coder yegor256 --coder John --verbose --dry --to foo"
+    When I run bin/cobench with "--coder yegor256 --coder Jeff --verbose --dry --to foo"
     Then Stdout contains "XML saved to"
     And Exit code is zero
 

data/lib/cobench/metrics/commits.rb

@@ -34,7 +34,7 @@ class Cobench::Commits
 
   def take(loog)
     from = (Time.now - (60 * 60 * 24 * @opts[:days])).strftime('%Y-%m-%d')
-    q = "author:#{@user} author-date:>#{from}"
+    q = "author:#{@user} author-date:>#{from} is:public merge:false"
     json = @api.search_commits(q)
     loog.debug("Found #{json.total_count} commits")
     hoc = 0
@@ -44,6 +44,7 @@ class Cobench::Commits
       next unless Cobench::Match.new(@opts, loog).matches?(repo)
       loog.debug("Including #{sha} in #{repo}")
       json = @api.commit(repo, sha)
+      next unless json
       hocs = json.stats.total
       loog.debug("Found #{hocs} HoC in #{sha}")
       hoc += hocs

data/lib/cobench/version.rb

@@ -23,5 +23,5 @@
 # Copyright:: Copyright (c) 2022 Yegor Bugayenko
 # License:: MIT
 module Cobench
-  VERSION = '0.0.17'.freeze
+  VERSION = '0.0.20'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cobench
 version: !ruby/object:Gem::Version
-  version: 0.0.17
+  version: 0.0.20
 platform: ruby
 authors:
 - Yegor Bugayenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-08-16 00:00:00.000000000 Z
+date: 2022-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: backtrace