maxjacobson/film_snob — pull request #17 by CarlosPlusPlus (2014-05-27 05:15:16+00:00)
Title: Update video_site.rb with more semantic Ruby code.

Here's my one quick comment / recommendation:

```ruby
# If you ever have something of the form:
if hash['value']
hash['value']
else
'Empty Condition'
end

# Condense to one line via || (or) operator!
hash['value'] || "Empty Condition" # => So much cleaner!

```
========================================

Well, not you can't say I didn't look at the code base. :)

Overall it looks good, will try to come up with some more viable feedback throughout the week. There may be places where logic can be abstracted // meta-programmed further.