Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Ryan Orr

Ryan Orr

Not a member

Ryan Orr's latest activity

Monday, March 21 2011
6534 [PATCH] Fixing inconsistencies with Time{WithZone}#{hash,eql?} was updated in Ruby on Rails
  • Ryan Orr
    Ryan Orr commented at 2:49 AM

    I too ran the code that you made and thought it made sense. I added the test that Pan Thomakos discussed in this updated patch.
6572 [ Patch ] Added some helpful comments for Mime Type register method was updated in Ruby on Rails
  • Ryan Orr
    Ryan Orr commented at 1:18 AM

    Looks good to me. +1
Saturday, March 19 2011
6588 Add Enumerable#uniq? was updated in Ruby on Rails
  • Ryan Orr
    Ryan Orr commented at 6:19 PM

    Took a look at the code in the last patch that you posted. I like the implementation but you were missing a test that checked a block that produced a result tha...