{"push_id":629216453,"size":13,"distinct_size":1,"ref":"refs/heads/master","head":"6d3d11dcaa864f81bcca6bf7613b66e853155d98","before":"c4b46fd9d818ed82e5fe7d828e1a3f6e01c9e761","commits":[{"sha":"556045650019d2b959259899fd5b4cf10e98c4b9","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Claiming issue 414\n\nrelates to #414\n\n$ ./script/start-issue 414\n\n[skip ci]","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/556045650019d2b959259899fd5b4cf10e98c4b9"},{"sha":"53e9d1d80212e7d97f5be1699408062ea4649e73","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Adding request_change_on_behalf_of action","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/53e9d1d80212e7d97f5be1699408062ea4649e73"},{"sha":"04062d198b8e8d36d511d55bf0a343e1e28b145c","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Creating initial form for requesting changes\n\nThis form is inadequate but is the first step as I work through what\nis required.","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/04062d198b8e8d36d511d55bf0a343e1e28b145c"},{"sha":"4ef6b463a975ae5795105456914c4adbca1a14c6","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Adding validations for on_behalf_of_collaborator","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/4ef6b463a975ae5795105456914c4adbca1a14c6"},{"sha":"d659180834f17440471d415aff11e672d282bd0b","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Replacing instance variable with attr_accessor\n\nInstance variables are great but they make extension a bit more\ndifficult.","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/d659180834f17440471d415aff11e672d282bd0b"},{"sha":"dde3a8bf2e82ba8f3209c704ce05890f40b077ac","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Adding a ComposableElements module\n\nFavoring explicit composition vs. the wibbly wobbly\n`ActiveSupport::Concern`","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/dde3a8bf2e82ba8f3209c704ce05890f40b077ac"},{"sha":"3cce37579df96b4ae3f783605f1ba1d191013cc7","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Leveraging on_behalf_of_collaborator\n\nFactoring out composed methods in preference for a composed set of\nrelated methods.","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/3cce37579df96b4ae3f783605f1ba1d191013cc7"},{"sha":"4a20003a2312ce3910ddb89952688d8e1cde6530","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Extracting private method\n\nI'm trying to keep the methods at the same level of abstraction. I\nfeel this better approaches that.","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/4a20003a2312ce3910ddb89952688d8e1cde6530"},{"sha":"43b94f750317f65b5f9180c452fdde1f87e3bb69","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Moving method for greater clarity\n\nPutting method definitions closer to their related methods","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/43b94f750317f65b5f9180c452fdde1f87e3bb69"},{"sha":"3c5cc19c39dac66d5c13624cd8ecb0fc24fe7388","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Finalizing Request change on behalf of\n\nThere are common methods that could be extracted to a service method,\nhowever this is a first step before I go all in on that extraction.","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/3c5cc19c39dac66d5c13624cd8ecb0fc24fe7388"},{"sha":"39ae4f571d131c15364921f2478da3cc1ab84a5e","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Appeasing rubocop through method extraction\n\nThis is the first step in refactoring as I'll be able to make a\nservice object that holds the given state.","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/39ae4f571d131c15364921f2478da3cc1ab84a5e"},{"sha":"15b56b22faa75ce5ccf3456c7916852c1eb900f3","author":{"email":"cf980b0cb529ff541807b289951c7805cfbdb5b6@gmail.com","name":"Jeremy Friesen"},"message":"Closing issue 414\n\nCloses #414\nRelated to #468\n\n$ ./script/close-issue 414","distinct":false,"url":"https://api.github.com/repos/ndlib/sipity/commits/15b56b22faa75ce5ccf3456c7916852c1eb900f3"},{"sha":"6d3d11dcaa864f81bcca6bf7613b66e853155d98","author":{"email":"71819a9b1a638e9141273f896be434a93da2ed82@nd.edu","name":"rbalekai"},"message":"Merge pull request #469 from ndlib/414-Add-a-Changes-are-Requested-on-Behalf-of-action\n\n414 add a changes are requested on behalf of action","distinct":true,"url":"https://api.github.com/repos/ndlib/sipity/commits/6d3d11dcaa864f81bcca6bf7613b66e853155d98"}]}