Eh, i discourage double negative

` unless email_settings[Rails.env].nil?` ===

`if  (!email_settings[Rails.env].nil?` ===

`if (email_settings[Rails.env]`

So now you have:

```ruby
email_settings = YAML::load(File.open("#{Rails.root.to_s}/config/email.yml"))
ActionMailer::Base.smtp_settings = email_settings[Rails.env] if email_settings[Rails.env]
```

But what's going on here?  Your `if` is testing whether something bad is happening -- a configuration file isn't properly set.  That's an exception (Refactor:  "REPLACE CONDITIONAL WITH ASSERTION").

```
email_settings = YAML::load(File.open("#{Rails.root.to_s}/config/email.yml"))
raise "Email settings for #{Rails.env} not found!  So sad!" unless (settings = email_settings[Rails.env]) # guard clause
ActionMailer::Base.smtp_settings = settings 
```

Or, for WTF are you thinking points....:)

```ruby
ActionMailer::Base.smtp_settings = Hash.new{|h,k| raise "definition not found for '#{k}'"}.tap{|h| h.merge!(YAML::load(File.open("#{Rails.root.to_s}/config/email.yml")))}
```

But that's perverse.