metric_fu 2.1.3.5 → 2.1.3.6

data/Gemfile

@@ -1,5 +1,9 @@
 source :rubygems
 
-gem 'rake', '<=0.9.2'
-
+gem 'rake', '<0.9.3'
+group :development, :test do
+  gem "rspec", "= 1.3.0"
+  gem "test-construct", ">= 1.2.0"
+  gem "googlecharts"
+end
 gemspec

data/HISTORY.md

@@ -1,3 +1,12 @@
+=== MetricFu 2.1.3.6 / 2013-01-02
+
+* Fixed bug that wasn't show stats or rails_best_practices graphs
+* Updated churn and rails_best_practices gems
+* Move the metrics code in the rake task into its own file
+* Remove executable metric_fu dependency on rake
+* TODO: some unclear dependency issues may make metrics in 1.9 crash, esp Flog, Flay, Stats
+
+
 === MetricFu 2.1.3.5 / 2013-01-01
 
 * Issue #35, Namespace MetricFu::Table. -Benjamin Fleischer

data/README.md

@@ -26,6 +26,11 @@ __Code Quality__
 
 This project runs [https://codeclimate.com/](https://codeclimate.com/)
 
+__Gem Dependencies__
+
+[![Dependency Status](https://gemnasium.com/metricfu/metric_fu.png)](https://gemnasium.com/metricfu/metric_fu)
+
+This project runs [https://gemnasium.com/metricfu](https://gemnasium.com/metricfu)
 
 ===============================================================================
 

data/Rakefile

@@ -1,4 +1,5 @@
 #!/usr/bin/env rake
+require 'bundler/setup'
 require 'bundler/gem_tasks'
 # $LOAD_PATH << '.'
 begin
@@ -15,10 +16,4 @@ end
 
 require File.expand_path File.join(File.dirname(__FILE__),'lib/metric_fu')
 
-MetricFu::Configuration.run do |config|
-  config.roodi    = config.roodi.merge(:roodi_config => 'config/roodi_config.yml')
-  config.churn    = { :start_date => "1 year ago", :minimum_churn_count => 10}
-  config.hotspots = { :start_date => "1 year ago", :minimum_churn_count => 10}
-end
-
 task :default => :spec

data/TODO.md

@@ -0,0 +1,42 @@
+# TODO list
+
+
+## Features
+
+* Look into removing rcov and churn, and adding
+  * https://github.com/metricfu/code_statistics
+  * brakeman https://github.com/metricfu/brakeman
+  * cane https://github.com/square/cane
+  * laser https://github.com/metricfu/laser
+* Add configurable logger to all output streams
+* Color code flog results with scale from: http://jakescruggs.blogspot.com/2008/08/whats-good-flog-score.html
+* Make running metric_fu on metric_fu less embarrassing
+* Load all gems at config time so you fail fast if one is missing
+
+## Testing
+
+* Determine how to test metric_fu against codebases that are not metric_fu, to ensure it works on most applications
+* Re-organize test files structure to align with changed structure of library files
+* Remove / Modify Devver code from the generators/hotspots_spec and base/hotspot_analzyer_spec
+
+## Bugs / Fixes
+
+* Fix occasional gem install metric_fu failures such as with ripper in Ruby 1.9
+* Fork roodi and correct the yaml
+* See https://github.com/metricfu/metric_fu/issues/2 about updating gems
+
+## Ruby 1.9 compatibility
+
+* Consider `RUBYOPT='-rpsych'` e.g. from https://github.com/jscruggs/metric_fu/pull/77
+* Look into using the sexp_processor for ruby parsing in brakeman https://github.com/presidentbeef/brakeman/blob/cdc85962d589fb37e37ed53333bb0b7bd913e028/lib/ruby_parser/bm_sexp_processor.rb
+
+## Misc
+
+* Determine if CodeIssue is used, else remove it
+* Update the homepage http://metric-fu.rubyforge.org/
+* Other intersting libraries to consider:
+  * https://github.com/vinibaggio/discover-unused-partials
+  * https://github.com/metricfu/heckle
+  * https://github.com/metricfu/gauntlet
+  * https://github.com/metricfu/repodepot-ruby
+  * https://github.com/eladmeidar/rails_indexes

data/bin/metric_fu

@@ -1,4 +1,2 @@
-gem 'rake', '<= 0.9.2'
-require 'rake'
 require File.expand_path(File.join(File.dirname(__FILE__),'..','lib','metric_fu'))
-Rake::Task['metrics:all'].invoke
+MetricFu::Run.new.run

data/lib/configuration.rb

@@ -138,7 +138,7 @@ module MetricFu
       @reek     = { :dirs_to_reek => @code_dirs,
                     :config_file_pattern => nil}
       @roodi    = { :dirs_to_roodi => @code_dirs,
-                    :roodi_config => nil}
+                    :roodi_config => nil }
       @saikuro  = { :output_directory => "#{@scratch_directory}/saikuro",
                     :input_directory => @code_dirs,
                     :cyclo => "",

data/lib/data_structures/line_numbers.rb

@@ -22,7 +22,7 @@ module MetricFu
       end
     rescue Exception => e
       #catch errors for files ruby_parser fails on
-      puts "RUBY PARSE FAILURE: #{e.class}\t#{e.message}\tCONTENT:#{contents.inspect}\SEXP:#{file_sexp.inspect}\tn#{e.backtrace}"
+      puts "RUBY PARSE FAILURE: #{e.class}\t#{e.message}\tSEXP:#{file_sexp.inspect}\n\tCONTENT:#{contents.inspect}\n\t#{e.backtrace}"
       @locations
     end
 

data/lib/load_files.rb

@@ -3,18 +3,17 @@ MetricFu.reporting_require { 'report' }
 MetricFu.metrics_require   { 'generator' }
 MetricFu.metrics_require   { 'graph' }
 MetricFu.reporting_require { 'graphs/grapher' }
-MetricFu.lib_require       { 'scoring_strategies' }
+MetricFu.metrics_require   { 'hotspots/analysis/scoring_strategies' }
 
+# Now load everything else that's in the directory
+Dir.glob(File.join(MetricFu.lib_dir, '*.rb')).each do |file|
+  require file
+end
 # prevent the task from being run multiple times.
 unless Rake::Task.task_defined? "metrics:all"
   # Load the rakefile so users of the gem get the default metric_fu task
   MetricFu.tasks_load 'metric_fu.rake'
 end
-
-# Now load everything else that's in the directory
-Dir.glob(File.join(MetricFu.lib_dir, '*.rb')).each do |file|
-  require file
-end
 Dir.glob(File.join(MetricFu.data_structures_dir, '**/*.rb')).each do |file|
   require file
 end

data/lib/metrics/base_template.rb

@@ -50,7 +50,7 @@ module MetricFu
     # @return String
     #   A file path
     def template(section)
-      if MetricFu::AVAILABLE_METRICS.include?(section) # expects a symbol
+      if MetricFu.metrics.include?(section) # expects a symbol
         File.join(template_dir(section.to_s), "#{section}.html.erb")
       else
         File.join(template_directory,  section.to_s + ".html.erb")

data/lib/{data_structures → metrics/hotspots/analysis}/code_issue.rb

@@ -1,6 +1,6 @@
 require 'delegate'
 
-[ 'hotspot_analyzer',
+[ 'hotspots/hotspot_analyzer',
   'flog/flog_hotspot',
   'saikuro/saikuro_hotspot',
   'churn/churn_hotspot',
@@ -8,9 +8,10 @@ require 'delegate'
   'flay/flay_hotspot'].each do |path|
   MetricFu.metrics_require { path }
 end
-%w(careful_array record).each do |path|
+%w(careful_array).each do |path|
   MetricFu.data_structures_require { path }
 end
+MetricFu.metrics_require   { "hotspots/analysis/record" }
 
 module MetricFu
   class CodeIssue < DelegateClass(MetricFu::Record) #DelegateClass(Ruport::Data::Record)

data/lib/{data_structures → metrics/hotspots/analysis}/grouping.rb

@@ -1,5 +1,5 @@
 %w(table).each do |path|
-  MetricFu.data_structures_require { path }
+  MetricFu.metrics_require   { "hotspots/analysis/#{path}" }
 end
 module MetricFu
   class Grouping

data/lib/{data_structures → metrics/hotspots/analysis}/table.rb

@@ -1,5 +1,5 @@
 %w(record code_issue).each do |path|
-  MetricFu.data_structures_require { path }
+  MetricFu.metrics_require   { "hotspots/analysis/#{path}" }
 end
 
 module MetricFu

data/lib/metrics/{hotspot_analyzer.rb → hotspots/hotspot_analyzer.rb}

@@ -1,5 +1,10 @@
-%w(record grouping).each do |path|
-  MetricFu.data_structures_require { path }
+require File.expand_path('analysis_error', MetricFu.errors_dir)
+MetricFu.data_structures_require { 'location' }
+%w(table record grouping ranking).each do |path|
+  MetricFu.metrics_require   { "hotspots/analysis/#{path}" }
+end
+%w(reek roodi flog churn saikuro flay stats rcov).each do |path|
+  MetricFu.metrics_require   { "#{path}/#{path}_hotspot" }
 end
 
 module MetricFu

data/lib/metrics/hotspots/hotspots.rb

@@ -1,3 +1,4 @@
+MetricFu.metrics_require   { 'hotspots/hotspot_analyzer' }
 module MetricFu
 
   class Hotspots < Generator

data/lib/metrics/rails_best_practices/rails_best_practices.rb

@@ -1,8 +1,8 @@
 module MetricFu
   class RailsBestPractices < Generator
-
+    VERSION = '0.10.1'
     def emit
-      @output = `rails_best_practices --without-color .`
+      @output = `rails_best_practices _#{VERSION}_ --without-color .`
     end
 
     def analyze

data/lib/metrics/roodi/roodi.rb

@@ -5,7 +5,7 @@ module MetricFu
       files_to_analyze = MetricFu.roodi[:dirs_to_roodi].map{|dir| Dir[File.join(dir, "**/*.rb")] }
       files = remove_excluded_files(files_to_analyze.flatten)
       config = MetricFu.roodi[:roodi_config] ? "-config=#{MetricFu.roodi[:roodi_config]}" : ""
-      @output = `roodi #{config} #{files.join(" ")}`
+      @output = `metric_fu-roodi #{config} #{files.join(" ")}`
     end
 
     def analyze

data/lib/run.rb

@@ -0,0 +1,54 @@
+MetricFu::Configuration.run do |config|
+  config.roodi    = config.roodi.merge(:roodi_config => "#{MetricFu.root_dir}/config/roodi_config.yml")
+  config.churn    = { :start_date => "1 year ago", :minimum_churn_count => 10}
+  config.hotspots = { :start_date => "1 year ago", :minimum_churn_count => 10}
+end
+module MetricFu
+  class Run
+    def initialize
+      STDOUT.sync = true
+      MetricFu::Configuration.run {}
+    end
+    def run
+      add_metrics
+      save_reports
+      save_graphs
+      display_results
+    end
+    def add_metrics
+      MetricFu.metrics.each {|metric|
+        mf_debug "** STARTING METRIC #{metric}"
+        MetricFu.report.add(metric)
+        mf_debug "** ENDING METRIC #{metric}"
+      }
+    end
+    def save_reports
+      mf_debug "** SAVING REPORT YAML OUTPUT TO #{MetricFu.base_directory}"
+      MetricFu.report.save_output(MetricFu.report.to_yaml,
+                                  MetricFu.base_directory,
+                                  "report.yml")
+      mf_debug "** SAVING REPORT DATA OUTPUT TO #{MetricFu.data_directory}"
+      MetricFu.report.save_output(MetricFu.report.to_yaml,
+                                  MetricFu.data_directory,
+                                  "#{Time.now.strftime("%Y%m%d")}.yml")
+      mf_debug "** SAVING TEMPLATIZED REPORT"
+      MetricFu.report.save_templatized_report
+    end
+    def save_graphs
+      mf_debug "** PREPARING TO GRAPH"
+      MetricFu.graphs.each {|graph|
+        mf_debug "** Graphing #{graph} with #{MetricFu.graph_engine}"
+        MetricFu.graph.add(graph, MetricFu.graph_engine)
+      }
+      mf_debug "** GENERATING GRAPH"
+      MetricFu.graph.generate
+    end
+    def display_results
+      if MetricFu.report.open_in_browser?
+        mf_debug "** OPENING IN BROWSER FROM #{MetricFu.output_directory}"
+        MetricFu.report.show_in_browser(MetricFu.output_directory)
+      end
+    end
+  end
+end
+

data/lib/tasks/metric_fu.rake

@@ -2,35 +2,6 @@ require 'rake'
 namespace :metrics do
   desc "Generate all metrics reports"
   task :all do
-    STDOUT.sync = true
-    MetricFu::Configuration.run {}
-    MetricFu.metrics.each {|metric|
-      mf_debug "** STARTING METRIC #{metric}"
-      MetricFu.report.add(metric)
-      mf_debug "** ENDING METRIC #{metric}"
-    }
-    mf_debug "** SAVING REPORT YAML OUTPUT TO #{MetricFu.base_directory}"
-    MetricFu.report.save_output(MetricFu.report.to_yaml,
-                                MetricFu.base_directory,
-                                "report.yml")
-    mf_debug "** SAVING REPORT DATA OUTPUT TO #{MetricFu.data_directory}"
-    MetricFu.report.save_output(MetricFu.report.to_yaml,
-                                MetricFu.data_directory,
-                                "#{Time.now.strftime("%Y%m%d")}.yml")
-    mf_debug "** SAVING TEMPLATIZED REPORT"
-    MetricFu.report.save_templatized_report
-
-    mf_debug "** PREPARING TO GRAPH"
-    MetricFu.graphs.each {|graph|
-      mf_debug "** Graphing #{graph} with #{MetricFu.graph_engine}"
-      MetricFu.graph.add(graph, MetricFu.graph_engine)
-    }
-    mf_debug "** GENERATING GRAPH"
-    MetricFu.graph.generate
-
-    if MetricFu.report.open_in_browser?
-      mf_debug "** OPENING IN BROWSER FROM #{MetricFu.output_directory}"
-      MetricFu.report.show_in_browser(MetricFu.output_directory)
-    end
+    MetricFu::Run.new.run
   end
 end

data/lib/version.rb

@@ -1,3 +1,3 @@
 module MetricFu
-  VERSION = "2.1.3.5"
+  VERSION = "2.1.3.6"
 end

data/metric_fu.gemspec

@@ -10,6 +10,7 @@ Gem::Specification.new do |s|
   s.homepage    = "http://github.com/metricfu/metric_fu"
   s.description = "Code metrics from Flog, Flay, Simplecov-RCov, Saikuro, Churn, Reek, Roodi, Rails' stats task and Rails Best Practices"
   s.authors     = ["Jake Scruggs", "Sean Soper", "Andre Arko", "Petrik de Heus", "Grant McInnes", "Nick Quaranto", "Édouard Brière", "Carl Youngblood", "Richard Huang", "Dan Mayer", "Benjamin Fleischer"]
+  s.rubyforge_project = 'metric_fu'
   s.required_ruby_version     = ">= 1.8.7"
   s.required_rubygems_version = ">= 1.3.6"
   s.files              = `git ls-files`.split($\)
@@ -22,12 +23,11 @@ Gem::Specification.new do |s|
     "flay"                  => ["= 1.2.1"],
     "flog"                  => ["= 2.3.0"],
     "reek"                  => ["= 1.2.12"],
-    "roodi"                 => ["= 2.1.0"],
-    "rails_best_practices"  => ["~> 0.6"],
-    "churn"                 => ["= 0.0.7"],
-    "sexp_processor"        => ["~> 3.0.3"], # required because of churn.
-    "chronic"               => ["= 0.2.3"], # required by churn
-    "main"                  => ["= 4.7.1"], # required by churn
+    "metric_fu-roodi"       => [">= 2.2.0"],
+    "rails_best_practices"  => ["~> 0.10.1"],
+    "churn"                 => ["= 0.0.25"],
+    "sexp_processor"        => ["~> 3.0.3"], # required because of churn, flog, reek 1.2.12, ripper_ruby_parser 0.0.8
+    # "ruby_parser"           => ["~> 2.3"], # required because of churn, flog, reek 1.2.12, ripper_ruby_parser 0.0.8
     "activesupport"         => [">= 2.0.0"], # ok
     # "syntax"                => ["= 1.0.0"],
     "coderay"               => [],
@@ -57,9 +57,5 @@ Gem::Specification.new do |s|
   end
 
 
-  s.add_development_dependency("rake", '<=0.9.2')
-  s.add_development_dependency("rspec", ["= 1.3.0"])
-  s.add_development_dependency("test-construct", [">= 1.2.0"])
-  s.add_development_dependency("googlecharts")
 end
 

data/spec/base/base_template_spec.rb

@@ -56,7 +56,7 @@ describe MetricFu::Template do
   describe "#template" do
     it 'should generate the filename of the template file' do
       section = mock('section')
-      section.should_receive(:to_s).and_return('section')
+      section.should_receive(:to_s).any_number_of_times.and_return('section')
       @template.should_receive(:template_directory).and_return('dir')
       result = @template.send(:template, section)
       result.should == "dir/section.html.erb"

data/spec/generators/rails_best_practices_spec.rb

@@ -5,7 +5,7 @@ describe RailsBestPractices do
     it "should gather the raw data" do
       MetricFu::Configuration.run {}
       practices = MetricFu::RailsBestPractices.new
-      practices.should_receive(:`).with("rails_best_practices --without-color .")
+      practices.should_receive(:`).with("rails_best_practices _#{MetricFu::RailsBestPractices::VERSION}_ --without-color .")
       practices.emit
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification 
 name: metric_fu
 version: !ruby/object:Gem::Version 
-  hash: 97
+  hash: 103
   prerelease: 
   segments: 
   - 2
   - 1
   - 3
-  - 5
-  version: 2.1.3.5
+  - 6
+  version: 2.1.3.6
 platform: ruby
 authors: 
 - Jake Scruggs
@@ -29,117 +29,114 @@ cert_chain: []
 date: 2013-01-02 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: bluff
-  prerelease: false
   requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - ">="
+    - - "="
       - !ruby/object:Gem::Version 
-        hash: 3
+        hash: 45
         segments: 
         - 0
-        version: "0"
-  type: :runtime
+        - 0
+        - 25
+        version: 0.0.25
+  prerelease: false
+  name: churn
   version_requirements: *id001
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: map
-  prerelease: false
   requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - "="
+    - - ">="
       - !ruby/object:Gem::Version 
-        hash: 39
+        hash: 3
         segments: 
-        - 6
-        - 2
         - 0
-        version: 6.2.0
-  type: :runtime
+        version: "0"
+  prerelease: false
+  name: bluff
   version_requirements: *id002
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: activesupport
-  prerelease: false
   requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - ">="
+    - - "="
       - !ruby/object:Gem::Version 
-        hash: 15
+        hash: 5
         segments: 
         - 2
-        - 0
-        - 0
-        version: 2.0.0
-  type: :runtime
+        - 2
+        - 1
+        version: 2.2.1
+  prerelease: false
+  name: fattr
   version_requirements: *id003
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: main
-  prerelease: false
   requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 33
+        hash: 39
         segments: 
-        - 4
-        - 7
-        - 1
-        version: 4.7.1
-  type: :runtime
+        - 6
+        - 2
+        - 0
+        version: 6.2.0
+  prerelease: false
+  name: map
   version_requirements: *id004
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: churn
-  prerelease: false
   requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - "="
+    - - ">="
       - !ruby/object:Gem::Version 
-        hash: 17
+        hash: 3
         segments: 
         - 0
-        - 0
-        - 7
-        version: 0.0.7
-  type: :runtime
+        version: "0"
+  prerelease: false
+  name: googlecharts
   version_requirements: *id005
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: rails_best_practices
-  prerelease: false
   requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - ~>
+    - - "="
       - !ruby/object:Gem::Version 
-        hash: 7
+        hash: 43
         segments: 
-        - 0
-        - 6
-        version: "0.6"
-  type: :runtime
+        - 4
+        - 7
+        - 4
+        version: 4.7.4
+  prerelease: false
+  name: arrayfields
   version_requirements: *id006
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: roodi
-  prerelease: false
   requirement: &id007 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 11
+        hash: 7
         segments: 
-        - 2
         - 1
-        - 0
-        version: 2.1.0
-  type: :runtime
+        - 2
+        - 12
+        version: 1.2.12
+  prerelease: false
+  name: reek
   version_requirements: *id007
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: googlecharts
-  prerelease: false
   requirement: &id008 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
@@ -149,138 +146,108 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  type: :runtime
+  prerelease: false
+  name: coderay
   version_requirements: *id008
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: chronic
-  prerelease: false
   requirement: &id009 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - "="
+    - - ">="
       - !ruby/object:Gem::Version 
-        hash: 17
+        hash: 15
         segments: 
-        - 0
         - 2
-        - 3
-        version: 0.2.3
-  type: :runtime
+        - 0
+        - 0
+        version: 2.0.0
+  prerelease: false
+  name: activesupport
   version_requirements: *id009
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: flay
-  prerelease: false
   requirement: &id010 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - "="
+    - - ~>
       - !ruby/object:Gem::Version 
-        hash: 29
+        hash: 1
         segments: 
-        - 1
-        - 2
-        - 1
-        version: 1.2.1
-  type: :runtime
+        - 3
+        - 0
+        - 3
+        version: 3.0.3
+  prerelease: false
+  name: sexp_processor
   version_requirements: *id010
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: fattr
-  prerelease: false
   requirement: &id011 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - "="
+    - - ~>
       - !ruby/object:Gem::Version 
-        hash: 5
+        hash: 53
         segments: 
-        - 2
-        - 2
+        - 0
+        - 10
         - 1
-        version: 2.2.1
-  type: :runtime
+        version: 0.10.1
+  prerelease: false
+  name: rails_best_practices
   version_requirements: *id011
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: flog
-  prerelease: false
   requirement: &id012 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - "="
+    - - ">="
       - !ruby/object:Gem::Version 
-        hash: 3
+        hash: 7
         segments: 
         - 2
-        - 3
+        - 2
         - 0
-        version: 2.3.0
-  type: :runtime
+        version: 2.2.0
+  prerelease: false
+  name: metric_fu-roodi
   version_requirements: *id012
+  type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: arrayfields
-  prerelease: false
   requirement: &id013 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
-      - !ruby/object:Gem::Version 
-        hash: 43
-        segments: 
-        - 4
-        - 7
-        - 4
-        version: 4.7.4
-  type: :runtime
-  version_requirements: *id013
-- !ruby/object:Gem::Dependency 
-  name: coderay
-  prerelease: false
-  requirement: &id014 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
       - !ruby/object:Gem::Version 
         hash: 3
         segments: 
-        - 0
-        version: "0"
-  type: :runtime
-  version_requirements: *id014
-- !ruby/object:Gem::Dependency 
-  name: sexp_processor
-  prerelease: false
-  requirement: &id015 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 1
-        segments: 
+        - 2
         - 3
         - 0
-        - 3
-        version: 3.0.3
+        version: 2.3.0
+  prerelease: false
+  name: flog
+  version_requirements: *id013
   type: :runtime
-  version_requirements: *id015
 - !ruby/object:Gem::Dependency 
-  name: reek
-  prerelease: false
-  requirement: &id016 !ruby/object:Gem::Requirement 
+  requirement: &id014 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 7
+        hash: 29
         segments: 
         - 1
         - 2
-        - 12
-        version: 1.2.12
+        - 1
+        version: 1.2.1
+  prerelease: false
+  name: flay
+  version_requirements: *id014
   type: :runtime
-  version_requirements: *id016
 - !ruby/object:Gem::Dependency 
-  name: ripper
-  prerelease: false
-  requirement: &id017 !ruby/object:Gem::Requirement 
+  requirement: &id015 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -291,12 +258,12 @@ dependencies:
         - 0
         - 5
         version: 1.0.5
+  prerelease: false
+  name: ripper
+  version_requirements: *id015
   type: :runtime
-  version_requirements: *id017
 - !ruby/object:Gem::Dependency 
-  name: rcov
-  prerelease: false
-  requirement: &id018 !ruby/object:Gem::Requirement 
+  requirement: &id016 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -306,12 +273,12 @@ dependencies:
         - 0
         - 8
         version: "0.8"
+  prerelease: false
+  name: rcov
+  version_requirements: *id016
   type: :runtime
-  version_requirements: *id018
 - !ruby/object:Gem::Dependency 
-  name: Saikuro
-  prerelease: false
-  requirement: &id019 !ruby/object:Gem::Requirement 
+  requirement: &id017 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -322,70 +289,10 @@ dependencies:
         - 1
         - 0
         version: 1.1.0
-  type: :runtime
-  version_requirements: *id019
-- !ruby/object:Gem::Dependency 
-  name: rake
-  prerelease: false
-  requirement: &id020 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - <=
-      - !ruby/object:Gem::Version 
-        hash: 63
-        segments: 
-        - 0
-        - 9
-        - 2
-        version: 0.9.2
-  type: :development
-  version_requirements: *id020
-- !ruby/object:Gem::Dependency 
-  name: rspec
   prerelease: false
-  requirement: &id021 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - "="
-      - !ruby/object:Gem::Version 
-        hash: 27
-        segments: 
-        - 1
-        - 3
-        - 0
-        version: 1.3.0
-  type: :development
-  version_requirements: *id021
-- !ruby/object:Gem::Dependency 
-  name: test-construct
-  prerelease: false
-  requirement: &id022 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 31
-        segments: 
-        - 1
-        - 2
-        - 0
-        version: 1.2.0
-  type: :development
-  version_requirements: *id022
-- !ruby/object:Gem::Dependency 
-  name: googlecharts
-  prerelease: false
-  requirement: &id023 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  type: :development
-  version_requirements: *id023
+  name: Saikuro
+  version_requirements: *id017
+  type: :runtime
 description: Code metrics from Flog, Flay, Simplecov-RCov, Saikuro, Churn, Reek, Roodi, Rails' stats task and Rails Best Practices
 email: github@benjaminfleischer.com
 executables: 
@@ -403,7 +310,7 @@ files:
 - Manifest.txt
 - README.md
 - Rakefile
-- TODO
+- TODO.md
 - bin/metric_fu
 - config/roodi_config.yml
 - home_page/back_all.jpg
@@ -425,18 +332,12 @@ files:
 - home_page/title_back.gif
 - lib/configuration.rb
 - lib/data_structures/careful_array.rb
-- lib/data_structures/code_issue.rb
-- lib/data_structures/grouping.rb
 - lib/data_structures/line_numbers.rb
 - lib/data_structures/location.rb
-- lib/data_structures/ranking.rb
-- lib/data_structures/record.rb
-- lib/data_structures/table.rb
 - lib/errors/analysis_error.rb
 - lib/initial_requires.rb
 - lib/load_files.rb
 - lib/logging/mf_debugger.rb
-- lib/md5_tracker.rb
 - lib/metric_fu.rb
 - lib/metrics/base_template.rb
 - lib/metrics/churn/churn.rb
@@ -455,7 +356,13 @@ files:
 - lib/metrics/flog/template_standard/flog.html.erb
 - lib/metrics/generator.rb
 - lib/metrics/graph.rb
-- lib/metrics/hotspot_analyzer.rb
+- lib/metrics/hotspots/analysis/code_issue.rb
+- lib/metrics/hotspots/analysis/grouping.rb
+- lib/metrics/hotspots/analysis/ranking.rb
+- lib/metrics/hotspots/analysis/record.rb
+- lib/metrics/hotspots/analysis/scoring_strategies.rb
+- lib/metrics/hotspots/analysis/table.rb
+- lib/metrics/hotspots/hotspot_analyzer.rb
 - lib/metrics/hotspots/hotspots.rb
 - lib/metrics/hotspots/template_awesome/hotspots.html.erb
 - lib/metrics/hotspots/template_standard/hotspots.html.erb
@@ -505,7 +412,7 @@ files:
 - lib/reporting/templates/standard/default.css
 - lib/reporting/templates/standard/index.html.erb
 - lib/reporting/templates/standard/standard_template.rb
-- lib/scoring_strategies.rb
+- lib/run.rb
 - lib/tasks/metric_fu.rake
 - lib/version.rb
 - metric_fu.gemspec
@@ -516,7 +423,6 @@ files:
 - spec/base/hotspot_analyzer_spec.rb
 - spec/base/line_numbers_spec.rb
 - spec/base/location_spec.rb
-- spec/base/md5_tracker_spec.rb
 - spec/base/ranking_spec.rb
 - spec/base/report_spec.rb
 - spec/base/table_spec.rb
@@ -583,7 +489,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.6
 requirements: []
 
-rubyforge_project: 
+rubyforge_project: metric_fu
 rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
@@ -596,7 +502,6 @@ test_files:
 - spec/base/hotspot_analyzer_spec.rb
 - spec/base/line_numbers_spec.rb
 - spec/base/location_spec.rb
-- spec/base/md5_tracker_spec.rb
 - spec/base/ranking_spec.rb
 - spec/base/report_spec.rb
 - spec/base/table_spec.rb

data/TODO

@@ -1,9 +0,0 @@
-== TODO list
-
-* Determine how to test metric_fu against codebases that are not metric_fu, to ensure it works on most applications
-* Add configurable logger to all output streams
-* Color code flog results with scale from: http://jakescruggs.blogspot.com/2008/08/whats-good-flog-score.html
-* Make running metric_fu on metric_fu less embarrassing
-* Load all gems at config time so you fail fast if one is missing
-* Look into removing rcov and churn, and adding laser, brakeman, and/or cane
-

data/lib/md5_tracker.rb

@@ -1,52 +0,0 @@
-require 'digest/md5'
-require 'fileutils'
-
-module MetricFu
-  class MD5Tracker
-
-    @@unchanged_md5s = []
-
-    class << self
-      def md5_dir(path_to_file, base_dir)
-        File.join(base_dir,
-                  path_to_file.split('/')[0..-2].join('/'))
-      end
-
-      def md5_file(path_to_file, base_dir)
-        File.join(md5_dir(path_to_file, base_dir),
-                  path_to_file.split('/').last.sub(/\.[a-z]+/, '.md5'))
-      end
-
-      def track(path_to_file, base_dir)
-        md5 = Digest::MD5.hexdigest(File.read(path_to_file))
-        FileUtils.mkdir_p(md5_dir(path_to_file, base_dir), :verbose => false)
-        f = File.new(md5_file(path_to_file, base_dir), "w")
-        f.puts(md5)
-        f.close
-        md5
-      end
-
-      def file_changed?(path_to_file, base_dir)
-        orig_md5_file = md5_file(path_to_file, base_dir)
-        return !!track(path_to_file, base_dir) unless File.exist?(orig_md5_file)
-
-        current_md5 = ""
-        file = File.open(orig_md5_file, 'r')
-        file.each_line { |line| current_md5 << line }
-        file.close
-        current_md5.chomp!
-
-        new_md5 = Digest::MD5.hexdigest(File.read(path_to_file))
-        new_md5.chomp!
-
-        @@unchanged_md5s << path_to_file if new_md5 == current_md5
-
-        return new_md5 != current_md5
-      end
-
-      def file_already_counted?(path_to_file)
-        return @@unchanged_md5s.include?(path_to_file)
-      end
-    end
-  end
-end

data/spec/base/md5_tracker_spec.rb

@@ -1,57 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
-
-describe MetricFu::MD5Tracker do
-  before do
-    @tmp_dir = File.join(File.dirname(__FILE__), 'tmp')
-    FileUtils.mkdir_p(@tmp_dir, :verbose => false) unless File.directory?(@tmp_dir)
-    @file1 = File.new(File.join(@tmp_dir, 'file1.txt'), 'w')
-    @file2 = File.new(File.join(@tmp_dir, 'file2.txt'), 'w')
-  end
-
-  after do
-    FileUtils.rm_rf(@tmp_dir, :verbose => false)
-  end
-
-  it "identical files should match" do
-    @file1.puts("Hello World")
-    @file1.close
-    file1_md5 = MD5Tracker.track(@file1.path, @tmp_dir)
-
-    @file2.puts("Hello World")
-    @file2.close
-    file2_md5 = MD5Tracker.track(@file2.path, @tmp_dir)
-
-    file2_md5.should == file1_md5
-  end
-
-  it "different files should not match" do
-    @file1.puts("Hello World")
-    @file1.close
-    file1_md5 = MD5Tracker.track(@file1.path, @tmp_dir)
-
-    @file2.puts("Goodbye World")
-    @file2.close
-    file2_md5 = MD5Tracker.track(@file2.path, @tmp_dir)
-
-    file2_md5.should_not == file1_md5
-  end
-
-  it "file_changed? should detect a change" do
-      @file2.close
-
-      @file1.puts("Hello World")
-      @file1.close
-      file1_md5 = MD5Tracker.track(@file1.path, @tmp_dir)
-
-      @file1 = File.new(File.join(@tmp_dir, 'file1.txt'), 'w')
-      @file1.puts("Goodbye World")
-      @file1.close
-      MD5Tracker.file_changed?(@file1.path, @tmp_dir).should be_true
-  end
-
-  it "should detect a new file" do
-    @file2.close
-    MD5Tracker.file_changed?(@file1.path, @tmp_dir).should be_true
-    File.exist?(MD5Tracker.md5_file(@file1.path, @tmp_dir)).should be_true
-  end
-end