Also If I am going to add these changes can I get some opinions on 

```
def send_mail(source)
  Mailer.deliver(
    to:      'bob@example.com',
    from:    'us@example.com',
    subject: 'Important message',
    body:    source.text)
end
```

vs

```
def send_mail(source)
  Mailer.deliver(
    to:      'bob@example.com',
    from:    'us@example.com',
    subject: 'Important message',
    body:    source.text
  )
end
```
with respect to the trailing bracket

Personally, I like the latter as you can use it consistently when nesting and it adds a bit of space in the code which can make things easier to read in some cases