Fixed the Rails version check. #35

merged 1 commit into from Jan 24, 2012


None yet

2 participants


The current version check is:

::Rails::VERSION::STRING.to_i >= 3.1

This fails because an integer has no decimal portion. Changing the to_i to to_f would work in this case, but it'll similarly lose the patch level portion of the version number. The check I'm doing in this pull request uses the same version comparison API that RubyGems uses itself.


Wow, that's pretty embarrassing. Thanks for the patch! :)

@jkrall jkrall merged commit a4dd831 into jkrall:master Jan 24, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment