GitHub PushEvent at 2014-06-19 20:14:00
Repository: griddler
Actor: calebthompson
Payload: {"shas":[["7cccd3b46291c4cd05fc103b8365a45b9efed51d","77544cb4d450f4c37e30115c8ae45f943c1f515e@calebthompson.io","Move process logic out of Griddler::Email\n\nThis is controller code that shouldn't have been in a data object to\nbegin with.","Caleb Thompson",true],["a432eeb196189cbeda1f3932b696250eb015edcf","77544cb4d450f4c37e30115c8ae45f943c1f515e@calebthompson.io","Use instances rather than class methods\n\nRather than sending Griddler.configuration.processor_method to\nGriddler.configuration.processor_class with a Griddler::Email as an\nargument (i.e., EmailProcessor.process(email)), instantiate\nGriddler.configuration.processor_class with a Griddler::Email and call\nGriddler.configuration.processor_method on that object (i.e.,\nEmailProcessor.new(email).process).\n\nThis encourages the behavior of separating logic into methods within an\nEmailProcessor, which is better than having a single class method with\nmany responsibilities or passing state around between class methods.","Caleb Thompson",true]],"size":2,"ref":"refs/heads/ct-move-process-to-controller","head":"a432eeb196189cbeda1f3932b696250eb015edcf"}