You don't need to change this, but I nearly always use an expression like `message.any?` here (rather than `messages.size > 0`).  Reason being, you can do `array.size > 0` in pretty much any language, but the `predicate?` syntax is unique to ruby and I always find it so readable...so I like to use it when I can since it's part of what I love about ruby!

...but as I said, what you have works fine :).