Skip to content

Commit

Permalink
Kill whitespace!
Browse files Browse the repository at this point in the history
  • Loading branch information
@drogus
drogus committed Dec 15, 2010
1 parent 232e379 commit 2dafb14
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions railties/lib/rails/generators/rails/app/app_generator.rb
View file
Original file line number Diff line number Diff line change
Expand Up @@ -104,18 +104,18 @@ def stylesheets

def javascripts
empty_directory "public/javascripts"

unless options[:skip_javascript]
copy_file "public/javascripts/#{@options[:javascript]}.js"
copy_file "public/javascripts/#{@options[:javascript]}_ujs.js", "public/javascripts/rails.js"

if options[:javascript] == "prototype"
copy_file "public/javascripts/controls.js"
copy_file "public/javascripts/dragdrop.js"
copy_file "public/javascripts/effects.js"
end
end

copy_file "public/javascripts/application.js"
end

Expand Down

2 comments on commit 2dafb14

@rymai
Copy link
Contributor

@rymai rymai commented on 2dafb14 Dec 15, 2010

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wondering, is there any convention for this?

I usually put new lines break like it was at lines 111 and 118, that way it goes directly to the right indentation (without hitting "tab") if a want to write a new line (and I also find it visually better).

@samuelkadolph
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The convention is no whitespace on empty lines. Most editors will indent for you automatically but remove the whitespace if you don't use the line.

Please sign in to comment.