Comments on: Rake task for syntax checking a Ruby on Rails project http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/ The guys behind LivingSocial Fri, 12 Mar 2010 02:58:37 +0000 http://wordpress.org/?v=2.9.2 hourly 1 By: Joe Grossberg http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-156 Joe Grossberg Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-156 Ooh, interesting. How would this approach work for CSS though, since it's not executed? Are you talking about using a W3C validator or the like? Ooh, interesting.

How would this approach work for CSS though, since it’s not executed? Are you talking about using a W3C validator or the like?

]]> By: Dan Manges http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-157 Dan Manges Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-157 Wouldn't it be much better to use tests? At least for the .rb and .erb files. Wouldn’t it be much better to use tests? At least for the .rb and .erb files.

]]> By: Chris Bailey http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-158 Chris Bailey Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-158 Pretty cool. I ran it on my current project, which uses Edge Rails/Rails 2.1, and every single error it spit out was from vendor/rails :) Pretty cool. I ran it on my current project, which uses Edge Rails/Rails 2.1, and every single error it spit out was from vendor/rails :)

]]> By: szeryf http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-159 szeryf Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-159 If you have Rails in vendor/rails, you might want to skip it in checking too. To do this, change the line 23 (3rd line of task :check_ruby) to: next if file.match("vendor/.*/generators/.*/templates") It also barfs on several Rails yml files, so I added the following before line 37 (3rd line of task :check_yml): next if file.match("vendor/rails") Other than that, it's really cool! :) If you have Rails in vendor/rails, you might want to skip it in checking too. To do this, change the line 23 (3rd line of task :check_ruby) to:

next if file.match("vendor/.*/generators/.*/templates")

It also barfs on several Rails yml files, so I added the following before line 37 (3rd line of task :check_yml):

next if file.match("vendor/rails")

Other than that, it’s really cool! :)

]]> By: Warren http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-160 Warren Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-160 Joe: I haven't found one yet, but I would imagine that there is a regular expression that describes valid syntax for a CSS file. Syntax checking would simply be a matter of checking each .css file against this regexp. Hitting the W3C service would most likely be very slow. Dan: Test don't necessarily cover every single file in your project. Chris/szeryf: I hadn't thought of skipping vendor/rails... great idea! I've updated the post to reflect that! Skipping vendor/rails should also speed this task up significantly. Joe: I haven’t found one yet, but I would imagine that there is a regular expression that describes valid syntax for a CSS file. Syntax checking would simply be a matter of checking each .css file against this regexp. Hitting the W3C service would most likely be very slow.

Dan: Test don’t necessarily cover every single file in your project.

Chris/szeryf: I hadn’t thought of skipping vendor/rails… great idea! I’ve updated the post to reflect that! Skipping vendor/rails should also speed this task up significantly.

]]> By: Adam Bair http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-161 Adam Bair Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-161 Using it now... Look at that, it's already found some ugly. I set it up to ignore vendor/plugins as well as vendor/rails. Thanks guys, nice work! Using it now… Look at that, it’s already found some ugly. I set it up to ignore vendor/plugins as well as vendor/rails. Thanks guys, nice work!

]]> By: anonymous http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-162 anonymous Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-162 Wouldn't it be better to just use flog or heckle? Wouldn’t it be better to just use flog or heckle?

]]> By: Michael Fairchild http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-163 Michael Fairchild Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-163 Thanks, on the first run I found an error in my project. Much better me than someone else. I'll be using this all the time now. Thanks, on the first run I found an error in my project. Much better me than someone else. I’ll be using this all the time now.

]]> By: Morten Christensen http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-164 Morten Christensen Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-164 RailsCheck (http://railscheck.rubyforge.org/) do all this and more. RailsCheck (http://railscheck.rubyforge.org/) do all this and more.

]]> By: Wikki http://blog.hungrymachine.com/2008/06/04/rake-task-for-syntax-checking-a-ruby-on-rails-project/comment-page-1/#comment-165 Wikki Wed, 04 Jun 2008 16:00:00 +0000 blog.hungrymachine.com/2008/06/05/rake-task-for-syntax-checking-a-ruby-on-rails-project#comment-165 Nice thing, I am going to use it... Nice thing, I am going to use it…

]]>