Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Clean up the output of rake stats, de-emphasise components and apis,…

… and remove the indents for tests [Koz]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@5369 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
commit e808315725d7dad1745572c8ef9ae7e4f6ff0b48 1 parent 464102d
@NZKoz NZKoz authored
View
2  railties/CHANGELOG
@@ -1,5 +1,7 @@
*SVN*
+* Clean up the output of rake stats, de-emphasise components and apis, and remove the indents for tests [Koz]
+
* Added option to script/process/spawner of specifying the binding address #5133 [zsombor]
* Update environment.rb comments to include config.autoload_paths. Closes #6478 [caio]
View
2  railties/lib/code_statistics.rb
@@ -83,7 +83,7 @@ def print_line(name, statistics)
loc_over_m = (statistics["codelines"] / statistics["methods"]) - 2 rescue loc_over_m = 0
start = if TEST_TYPES.include? name
- "| #{name.ljust(18)} "
+ "| #{name.ljust(20)} "
else
"| #{name.ljust(20)} "
end
View
11 railties/lib/tasks/statistics.rake
@@ -1,13 +1,14 @@
STATS_DIRECTORIES = [
+ %w(Controllers app/controllers),
%w(Helpers app/helpers),
- %w(Controllers app/controllers),
+ %w(Models app/models),
+ %w(Libraries lib/),
%w(APIs app/apis),
%w(Components components),
+ %w(Integration\ tests test/integration),
%w(Functional\ tests test/functional),
- %w(Models app/models),
- %w(Unit\ tests test/unit),
- %w(Libraries lib/),
- %w(Integration\ tests test/integration)
+ %w(Unit\ tests test/unit)
+
].collect { |name, dir| [ name, "#{RAILS_ROOT}/#{dir}" ] }.select { |name, dir| File.directory?(dir) }
desc "Report code statistics (KLOCs, etc) from the application"

1 comment on commit e808315

@kleb

This commit seems to have rendered the start variable superfluous, i.e., both branches of the conditional assignment are identical?

Please sign in to comment.
Something went wrong with that request. Please try again.