rails_best_practices 0.6.5 → 0.6.6

data/MIT_LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2009 - 2010 Richard Huang (flyerhzm@gmail.com)
+Copyright (c) 2009 - 2011 Richard Huang (flyerhzm@gmail.com)
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -3,11 +3,6 @@ rails_best_practices
 
 rails_best_practices is a code metric tool to check the quality of rails codes.
 
-Donate
-------
-
-<a href='http://www.pledgie.com/campaigns/12057'><img alt='Click here to lend your support to: rails-bestpractices.com and make a donation at www.pledgie.com !' src='http://www.pledgie.com/campaigns/12057.png?skin_name=chrome' border='0' /></a>
-
 Usage
 -----
 
@@ -26,6 +21,7 @@ By default rails_best_practices will do parse codes in vendor, spec, test and fe
         -d, --debug                      Debug mode
         -f, --format FORMAT              output format
             --without-color              only output plain text without color
+            --with-textmate              open file by textmate in html format
             --vendor                     include vendor files
             --spec                       include spec files
             --test                       include test files
@@ -53,6 +49,11 @@ Wiki: <http://github.com/flyerhzm/rails_best_practices/wiki>
 
 Issue Tracker: <http://github.com/flyerhzm/rails_best_practices/issues>
 
+Donating
+--------
+
+<a href='http://www.pledgie.com/campaigns/12057'>Support rails_best_practices at Pledgie</a>
+
 Install
 -------
 
@@ -74,7 +75,7 @@ First run
 
     rails_best_practices -g
 
-to generate <code>rails_best_practices.yml</code> file.
+to generate `rails_best_practices.yml` file.
 
 Now you can customize this configuration file, the default configuration is as follows:
 
@@ -164,4 +165,4 @@ Follow us on twitter: <http://twitter.com/railsbp>
 Send us email: <team@rails-bestpractices.com>
 
 
-Copyright © 2010 Richard Huang (flyerhzm@gmail.com), released under the MIT license
+Copyright © 2009 - 2011 Richard Huang (flyerhzm@gmail.com), released under the MIT license

data/assets/result.html.haml

@@ -35,7 +35,10 @@
       }
   %body
     %h1 rails_best_practices output
-    %h2 Please go to <a href="http://rails-bestpractices.com">http://rails-bestpractices.com</a> to see more useful Rails Best Practices.
+    %h2
+      Please go to
+      %a{:href=>"http://rails-bestpractices.com", :target=>"_blank"} http://rails-bestpractices.com
+      to see more useful Rails Best Practices.
     %h2
       - if errors.empty?
         No error found. Cool!
@@ -48,6 +51,11 @@
         %th Error Message
       - errors.each do |error|
         %tr
-          %td.filename= error.filename
+          %td.filename
+            - if textmate
+              %a{:href=>"txmt://open/?url=file://#{File.expand_path(error.filename)}&line=#{error.line_number}"}= error.filename
+            - else
+              = error.filename
           %td.line-number= error.line_number
-          %td.message= error.message
+          %td.message
+            %a{:href=>"#{error.url}", :target=>"_blank"}= error.message

data/autotest/discover.rb

@@ -0,0 +1 @@
+Autotest.add_discovery { "rspec2" }

data/install_supported_rubies.sh

@@ -0,0 +1,8 @@
+#!/bin/bash
+rubies=( 1.8.7 1.9.2 ree )
+for x in ${rubies[*]}
+do
+  rvm install $x || exit $?
+  rvm $x && rvm gemset use global && gem install bundler || exit $?
+  bundle install || exit $?
+done

data/lib/rails_best_practices.rb

@@ -28,6 +28,7 @@ require 'colored'
 require 'haml'
 require 'rails_best_practices/checks'
 require 'rails_best_practices/core'
+require 'fileutils'
 
 # RailsBestPractices helps you to analyze your rails code, according to best practices on http://rails-bestpractices.
 # if it finds any violatioins to best practices, it will give you some readable suggestions.
@@ -70,10 +71,11 @@ module RailsBestPractices
       Core::Runner.base_path = @path
       @runner = Core::Runner.new
       @runner.debug = true if @options['debug']
+      @runner.color = !options['without-color']
 
       if @runner.checks.find { |check| check.is_a? Checks::AlwaysAddDbIndexCheck } &&
          !review_files.find { |file| file.index "db\/schema.rb" }
-        puts "AlwaysAddDbIndexCheck is disabled as there is no db/schema.rb file in your rails project.".blue
+        plain_output("AlwaysAddDbIndexCheck is disabled as there is no db/schema.rb file in your rails project.", 'blue')
       end
 
       @bar = ProgressBar.new('Analyzing', prepare_files.size + review_files.size)
@@ -139,7 +141,7 @@ module RailsBestPractices
     # @param [Array] dirs what directories to expand
     # @return [Array] all files expanded
     def expand_dirs_to_files *dirs
-      extensions = ['rb', 'erb', 'haml', 'builder']
+      extensions = ['rb', 'erb', 'rhtml', 'haml', 'builder']
 
       dirs.flatten.map { |entry|
         next unless File.exist? entry
@@ -206,7 +208,7 @@ module RailsBestPractices
       template = File.read(File.join(File.dirname(__FILE__), "..", "assets", "result.html.haml"))
 
       File.open("rails_best_practices_output.html", "w+") do |file|
-        file.puts Haml::Engine.new(template).render(Object.new, :errors => @runner.errors)
+        file.puts Haml::Engine.new(template).render(Object.new, :errors => @runner.errors, :textmate => @options["with-textmate"])
       end
     end
   end

data/lib/rails_best_practices/checks/add_model_virtual_attribute_check.rb

@@ -27,6 +27,9 @@ module RailsBestPractices
     #
     #   then the model needs to add a virtual attribute.
     class AddModelVirtualAttributeCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/4-add-model-virtual-attribute"
+      end
 
       def interesting_review_nodes
         [:defn]

data/lib/rails_best_practices/checks/always_add_db_index_check.rb

@@ -27,6 +27,9 @@ module RailsBestPractices
     #   if there are any foreign keys not existed in index columns,
     #   then the foreign keys should add db index.
     class AlwaysAddDbIndexCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/21-always-add-db-index"
+      end
 
       def interesting_review_nodes
         [:call, :iter]

data/lib/rails_best_practices/checks/check.rb

@@ -137,7 +137,7 @@ module RailsBestPractices
       #   file is the filename of source code
       #   line is the line number of the source code which is reviewing
       def add_error(error, file = @node.file, line = @node.line)
-        @errors << RailsBestPractices::Core::Error.new("#{file}", "#{line}", error)
+        @errors << RailsBestPractices::Core::Error.new("#{file}", "#{line}", error, url)
       end
 
       # remember use count for the local or instance variable in the call or attrasgn node.

data/lib/rails_best_practices/checks/dry_bundler_in_capistrano_check.rb

@@ -18,6 +18,9 @@ module RailsBestPractices
     #   if the message of call node is :namespace and the arguments of the call node is :bundler,
     #   then it should use bundler's capistrano recipe.
     class DryBundlerInCapistranoCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/51-dry-bundler-in-capistrano"
+      end
 
       def interesting_review_nodes
         [:call]

data/lib/rails_best_practices/checks/isolate_seed_data_check.rb

@@ -24,6 +24,9 @@ module RailsBestPractices
     #   and the subject is included in new variables,
     #   then it should be isolated to db seed.
     class IsolateSeedDataCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/20-isolating-seed-data"
+      end
 
       def interesting_review_nodes
         [:call, :lasgn, :iasgn]

data/lib/rails_best_practices/checks/keep_finders_on_their_own_model_check.rb

@@ -23,6 +23,10 @@ module RailsBestPractices
 
       FINDERS = [:find, :all, :first, :last]
 
+      def url
+        "http://rails-bestpractices.com/posts/13-keep-finders-on-their-own-model"
+      end
+
       def interesting_review_nodes
         [:call]
       end

data/lib/rails_best_practices/checks/law_of_demeter_check.rb

@@ -22,6 +22,10 @@ module RailsBestPractices
 
       prepare_model_associations
 
+      def url
+        "http://rails-bestpractices.com/posts/15-the-law-of-demeter"
+      end
+
       def interesting_review_nodes
         [:call]
       end

data/lib/rails_best_practices/checks/move_code_into_controller_check.rb

@@ -18,6 +18,10 @@ module RailsBestPractices
 
       FINDERS = [:find, :all, :first, :last]
 
+      def url
+        "http://rails-bestpractices.com/posts/24-move-code-into-controller"
+      end
+
       def interesting_review_nodes
         [:call]
       end

data/lib/rails_best_practices/checks/move_code_into_helper_check.rb

@@ -22,6 +22,9 @@ module RailsBestPractices
     #   and the size of the array is greater than array_count defined,
     #   then the options_for_select method should be moved into helper.
     class MoveCodeIntoHelperCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/26-move-code-into-helper"
+      end
 
       def interesting_review_nodes
         [:call]

data/lib/rails_best_practices/checks/move_code_into_model_check.rb

@@ -17,6 +17,9 @@ module RailsBestPractices
     #   and the subject is a local variable or instance variable,
     #   then they should be moved into model.
     class MoveCodeIntoModelCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/25-move-code-into-model"
+      end
 
       def interesting_review_nodes
         [:if]

data/lib/rails_best_practices/checks/move_finder_to_named_scope_check.rb

@@ -21,6 +21,10 @@ module RailsBestPractices
 
       FINDER = [:find, :all, :first, :last]
 
+      def url
+        "http://rails-bestpractices.com/posts/1-move-finder-to-named_scope"
+      end
+
       def interesting_review_nodes
         [:call]
       end

data/lib/rails_best_practices/checks/move_model_logic_into_model_check.rb

@@ -18,6 +18,9 @@ module RailsBestPractices
     #   and the subject is a local variable or an instance variable,
     #   then they are complex model logic, and they should be moved into model.
     class MoveModelLogicIntoModelCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/7-move-model-logic-into-the-model"
+      end
 
       def interesting_review_nodes
         [:defn]

data/lib/rails_best_practices/checks/needless_deep_nesting_check.rb

@@ -30,6 +30,9 @@ module RailsBestPractices
     #   and the @counter is greater than @nested_count defined,
     #   then it is a needless deep nesting.
     class NeedlessDeepNestingCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/11-needless-deep-nesting"
+      end
 
       def interesting_review_nodes
         [:call, :iter]

data/lib/rails_best_practices/checks/not_use_default_route_check.rb

@@ -22,6 +22,9 @@ module RailsBestPractices
     #
     #     match ':controller(/:action(/:id(.:format)))'
     class NotUseDefaultRouteCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/12-not-use-default-route-if-you-use-restful-design"
+      end
 
       def interesting_review_nodes
         [:call]

data/lib/rails_best_practices/checks/overuse_route_customizations_check.rb

@@ -34,6 +34,10 @@ module RailsBestPractices
 
       VERBS = [:get, :post, :update, :delete]
 
+      def url
+        "http://rails-bestpractices.com/posts/10-overuse-route-customizations"
+      end
+
       def interesting_review_nodes
         [:call, :iter]
       end

data/lib/rails_best_practices/checks/replace_complex_creation_with_factory_method_check.rb

@@ -19,6 +19,9 @@ module RailsBestPractices
     #   and after them there is a call node with message :save or :save!,
     #   then these attribute assignments are complex creation, should be replaced with factory method.
     class ReplaceComplexCreationWithFactoryMethodCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/6-replace-complex-creation-with-factory-method"
+      end
 
       def interesting_review_nodes
         [:defn]

data/lib/rails_best_practices/checks/replace_instance_variable_with_local_variable_check.rb

@@ -16,6 +16,9 @@ module RailsBestPractices
     #   check all instance variable in partial view files,
     #   if exist, then they should be replaced with local variable
     class ReplaceInstanceVariableWithLocalVariableCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/27-replace-instance-variable-with-local-variable"
+      end
 
       def interesting_review_nodes
         [:ivar]

data/lib/rails_best_practices/checks/use_before_filter_check.rb

@@ -17,6 +17,12 @@ module RailsBestPractices
     #   if they are duplicated, then they should be moved to before_filter.
     class UseBeforeFilterCheck < Check
 
+      PROTECTED_PRIVATE_CALLS = [[:call, nil, :protected, [:arglist]], [:call, nil, :private, [:arglist]]]
+
+      def url
+        "http://rails-bestpractices.com/posts/22-use-before_filter"
+      end
+
       def interesting_review_nodes
         [:class]
       end
@@ -27,7 +33,7 @@ module RailsBestPractices
 
       # check class define node to see if there are method define nodes whose first code line are duplicated in review process.
       #
-      # it will every defn nodes in the class node,
+      # it will check every defn nodes in the class node until protected or private identification,
       # if there are defn nodes who have the same first code line, like
       #
       #     s(:class, :PostsController, s(:const, :ApplicationController),
@@ -70,7 +76,11 @@ module RailsBestPractices
       # then these duplicated first code lines should be moved to before_filter.
       def review_start_class(class_node)
         @first_sentences = {}
-        class_node.grep_nodes({:node_type => :defn}) { |defn_node| remember_first_sentence(defn_node) }
+
+        class_node.body.children.each do |child_node|
+          break if PROTECTED_PRIVATE_CALLS.include? child_node
+          remember_first_sentence(child_node) if :defn == child_node.node_type
+        end
         @first_sentences.each do |first_sentence, defn_nodes|
           if defn_nodes.size > 1
             add_error "use before_filter for #{defn_nodes.collect(&:method_name).join(',')}", class_node.file, defn_nodes.collect(&:line).join(',')

data/lib/rails_best_practices/checks/use_model_association_check.rb

@@ -19,6 +19,9 @@ module RailsBestPractices
     #   and the subjects of attribute assignment node and call node are the same,
     #   then model association should be used instead of xxx_id assignment.
     class UseModelAssociationCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/2-use-model-association"
+      end
 
       def interesting_review_nodes
         [:defn]

data/lib/rails_best_practices/checks/use_observer_check.rb

@@ -24,6 +24,9 @@ module RailsBestPractices
     #   and there is a mailer deliver call,
     #   then the method should be replaced by using observer.
     class UseObserverCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/19-use-observer"
+      end
 
       def interesting_prepare_nodes
         [:class]

data/lib/rails_best_practices/checks/use_query_attribute_check.rb

@@ -26,6 +26,10 @@ module RailsBestPractices
 
       prepare_model_associations
 
+      def url
+        "http://rails-bestpractices.com/posts/56-use-query-attribute"
+      end
+
       def interesting_review_nodes
         [:if]
       end

data/lib/rails_best_practices/checks/use_say_with_time_in_migrations_check.rb

@@ -22,6 +22,9 @@ module RailsBestPractices
       DEFAULT_MIGRATION_METHODS = [:add_column, :add_index, :add_timestamps, :change_column, :change_column_default, :change_table, :create_table, :drop_table, :remove_column, :remove_index, :remove_timestamps, :rename_column, :rename_index, :rename_table]
       WITH_SAY_METHODS = DEFAULT_MIGRATION_METHODS + [:say, :say_with_time]
 
+      def url
+        "http://rails-bestpractices.com/posts/46-use-say-and-say_with_time-in-migrations-to-make-a-useful-migration-log"
+      end
 
       def interesting_review_nodes
         [:defs]

data/lib/rails_best_practices/checks/use_scope_access_check.rb

@@ -19,6 +19,9 @@ module RailsBestPractices
     #   and there is redirect_to method call in if block body,
     #   then it should be replaced by using scope access.
     class UseScopeAccessCheck < Check
+      def url
+        "http://rails-bestpractices.com/posts/3-use-scope-access"
+      end
 
       def interesting_review_nodes
         [:if]

data/lib/rails_best_practices/command.rb

@@ -2,17 +2,19 @@
 require 'optparse'
 
 # Usage: rails_best_practices [options] path
-#     -d, --debug                      Debug mode
-#     -f, --format FORMAT              output format
-#         --vendor                     include vendor files
-#         --spec                       include spec files
-#         --test                       include test files
-#         --features                   include features files
-#     -x, --exclude PATTERNS           Don't analyze files matching a pattern
-#                                      (comma-separated regexp list)
-#     -g, --generate                   Generate configuration yaml
-#     -v, --version                    Show this version
-#     -h, --help                       Show this message
+#    -d, --debug                      Debug mode
+#    -f, --format FORMAT              output format
+#        --without-color              only output plain text without color
+#        --with-textmate              open file by textmate in html format
+#        --vendor                     include vendor files
+#        --spec                       include spec files
+#        --test                       include test files
+#        --features                   include features files
+#    -x, --exclude PATTERNS           Don't analyze files matching a pattern
+#                                     (comma-separated regexp list)
+#    -g, --generate                   Generate configuration yaml
+#    -v, --version                    Show this version
+#    -h, --help                       Show this message
 options = {}
 OptionParser.new do |opts|
   opts.banner = "Usage: rails_best_practices [options] path"
@@ -29,6 +31,10 @@ OptionParser.new do |opts|
     options["without-color"] = true
   end
 
+  opts.on("--with-textmate", "open file by textmate in html format") do
+    options["with-textmate"] = true
+  end
+
   ['vendor', 'spec', 'test', 'features'].each do |pattern|
     opts.on("--#{pattern}", "include #{pattern} files") do
       options[pattern] = true

data/lib/rails_best_practices/core/error.rb

@@ -5,12 +5,13 @@ module RailsBestPractices
     #
     # it indicates the filenname, line number and error message for the violation.
     class Error
-      attr_reader :filename, :line_number, :message
+      attr_reader :filename, :line_number, :message, :url
 
-      def initialize(filename, line_number, message)
+      def initialize(filename, line_number, message, url = nil)
         @filename = filename
         @line_number = line_number
         @message = message
+        @url = url
       end
 
       def to_s

data/lib/rails_best_practices/core/runner.rb

@@ -15,7 +15,7 @@ module RailsBestPractices
     # 2. review process, it does real check, if the source code violates some best practices, the violations will be notified.
     class Runner
       attr_reader :checks
-      attr_accessor :debug
+      attr_accessor :debug, :color
 
       # set the base path.
       #
@@ -78,7 +78,11 @@ module RailsBestPractices
           begin
             RubyParser.new.parse(content, filename)
           rescue Exception => e
-            puts "#{filename} looks like it's not a valid Ruby file.  Skipping...".red if @debug
+            if @debug
+              warning = "#{filename} looks like it's not a valid Ruby file.  Skipping..."
+              warning = warning.red if self.color
+              puts warning
+            end
             nil
           end
         end
@@ -88,7 +92,7 @@ module RailsBestPractices
         # filename is the filename of the erb or haml code.
         # content is the source code of erb or haml file.
         def parse_erb_or_haml(filename, content)
-          if filename =~ /.*\.erb$/
+          if filename =~ /.*\.erb|.*\.rhtml$/
             content = Erubis::Eruby.new(content).src
           elsif filename =~ /.*\.haml$/
             begin

data/lib/rails_best_practices/version.rb

@@ -1,5 +1,5 @@
 # encoding: utf-8
 module RailsBestPractices
-  VERSION = "0.6.5"
+  VERSION = "0.6.6"
 end
 

data/rails_best_practices.gemspec

@@ -29,4 +29,20 @@ Gem::Specification.new do |s|
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
+
+  s.post_install_message = <<-POST_INSTALL_MESSAGE
+#{"*" * 80}
+
+  rails_best_practices is a code metric tool to check the quality of rails codes.
+
+  I highly recommend you go through the Rails Best Practices website first.
+
+      http://rails-bestpractices.com
+
+  Enjoy!
+
+      Richard Huang (flyerhzm@gmail.com)
+
+#{"*" * 80}
+  POST_INSTALL_MESSAGE
 end

data/rake_rubies.sh

@@ -0,0 +1,9 @@
+#!/bin/bash
+rubies=( 1.8.7 1.9.2 ree )
+gemset="rails_best_practices"
+
+for x in ${rubies[*]}
+do
+  rvm use --create $x@$gemset && bundle install || exit $?
+done
+rvm ree@$gemset,1.8.7@$gemset,1.9.2@$gemset rake spec:progress

data/spec/rails_best_practices/checks/use_before_filter_check_spec.rb

@@ -82,4 +82,24 @@ describe RailsBestPractices::Checks::UseBeforeFilterCheck do
     errors = @runner.errors
     errors.should be_empty
   end
+
+  it "should not use before_filter for protected/private methods" do
+    content =<<-EOF
+    class PostsController < ApplicationController
+      protected
+        def load_comments
+          load_post
+          @comments = @post.comments
+        end
+
+        def load_user
+          load_post
+          @user = @post.user
+        end
+    end
+    EOF
+    @runner.review('app/controllers/posts_controller.rb', content)
+    errors = @runner.errors
+    errors.should be_empty
+  end
 end

data/spec/rails_best_practices_spec.rb

@@ -26,7 +26,7 @@ describe RailsBestPractices do
     end
   end
 
-  describe "output_errors" do
+  describe "output_terminal_errors" do
     check1 = RailsBestPractices::Checks::LawOfDemeterCheck.new
     check2 = RailsBestPractices::Checks::UseQueryAttributeCheck.new
     runner = RailsBestPractices::Core::Runner.new(check1, check2)
@@ -36,7 +36,7 @@ describe RailsBestPractices do
 
     $origin_stdout = $stdout
     $stdout = StringIO.new
-    RailsBestPractices.output_errors
+    RailsBestPractices.output_terminal_errors
     result = $stdout.string
     $stdout = $origin_stdout
     result.should == ["app/models/user.rb:10 - law of demeter".red, "app/models/post.rb:100 - use query attribute".red, "\nPlease go to http://rails-bestpractices.com to see more useful Rails Best Practices.".green, "\nFound 2 errors.".red].join("\n") + "\n"

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: rails_best_practices
 version: !ruby/object:Gem::Version 
-  hash: 13
+  hash: 11
   prerelease: false
   segments: 
   - 0
   - 6
-  - 5
-  version: 0.6.5
+  - 6
+  version: 0.6.6
 platform: ruby
 authors: 
 - Richard Huang
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-04 00:00:00 +08:00
+date: 2011-01-24 00:00:00 +08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -192,7 +192,9 @@ files:
 - README.md
 - Rakefile
 - assets/result.html.haml
+- autotest/discover.rb
 - bin/rails_best_practices
+- install_supported_rubies.sh
 - lib/rails_best_practices.rb
 - lib/rails_best_practices/checks.rb
 - lib/rails_best_practices/checks/add_model_virtual_attribute_check.rb
@@ -229,7 +231,7 @@ files:
 - lib/rails_best_practices/version.rb
 - rails_best_practices.gemspec
 - rails_best_practices.yml
-- rubies_test.sh
+- rake_rubies.sh
 - spec/rails_best_practices/checks/add_model_virtual_attribute_check_spec.rb
 - spec/rails_best_practices/checks/always_add_db_index_check_spec.rb
 - spec/rails_best_practices/checks/check_spec.rb
@@ -263,7 +265,21 @@ has_rdoc: true
 homepage: http://rails-bestpractices.com
 licenses: []
 
-post_install_message: 
+post_install_message: |
+  ********************************************************************************
+  
+    rails_best_practices is a code metric tool to check the quality of rails codes.
+  
+    I highly recommend you go through the Rails Best Practices website first.
+  
+        http://rails-bestpractices.com
+  
+    Enjoy!
+  
+        Richard Huang (flyerhzm@gmail.com)
+  
+  ********************************************************************************
+
 rdoc_options: []
 
 require_paths: 

data/rubies_test.sh

@@ -1,10 +0,0 @@
-#!/bin/bash
-rvm ruby-1.8.7@rails_best_practices ruby -S bundle install      || exit $?
-rvm ruby-1.9.2@rails_best_practices ruby -S bundle install      || exit $?
-rvm ree@rails_best_practices ruby -S bundle install             || exit $?
-rvm 1.8.7@rails_best_practices
-rake spec:progress
-rvm 1.9.2@rails_best_practices
-rake spec:progress
-rvm ree@rails_best_practices
-rake spec:progress