It's possible, but why do more work like that? I find that when the trailing comma is omitted, my eye is drawn to that line ending as if it were a ternary or camelCased. I miss it when it's gone :)

On Sat, Mar 2, 2013 at 11:27 AM, Dan Croak <notifications@github.com>
wrote:

>> @@ -29,5 +29,21 @@ def processor_class
>>      def reply_delimiter
>>        @reply_delimiter ||= 'Reply ABOVE THIS LINE'
>>      end
>> +
>> +    def email_service
>> +      @email_service_adapter ||= adapter_class[:sendgrid]
>> +    end
>> +
>> +    def email_service=(new_email_service)
>> +      @email_service_adapter = adapter_class.fetch(new_email_service) { raise Griddler::Errors::EmailServiceAdapterNotFound }
>> +    end
>> +
>> +    private
>> +
>> +    def adapter_class
>> +      {
>> +        sendgrid: Griddler::Adapters::SendgridAdapter,
>> +      }
> Neat, I didn't know that was possible.
> Would dropping to a word diff from the default unified diff when reviewing this part of code solve the same issue? I wonder if there's a GitHub web UI shortcut key to toggle that diff style.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/thoughtbot/griddler/pull/25/files#r3215319