engineyard/engineyard — pull request #175 by gtd (2013-12-07 12:33:37+00:00)
Title: Legacy ey.yml crashes `ey environment` opaquely

My `ey.yml` contained nil environment definitions. This results in the very opaque and untraceable (side note: could `ey` add a `--trace` option?):

undefined method `[]' for nil:NilClass

I don't know whether this is our fault or due to something in ancient EY history (we've been on the platform since early 2010, so we have some cruft). But in any case, it's very easy to end up in this situation if you delete the config for an environment but forget to delete the key. Then you end up with this very hard to trace error (it took me a half hour to figure out what was going on), so I propose this is an improvement all around.