{"action":"created","issue":{"url":"https://api.github.com/repos/tyarkoni/transitions/issues/80","repository_url":"https://api.github.com/repos/tyarkoni/transitions","labels_url":"https://api.github.com/repos/tyarkoni/transitions/issues/80/labels{/name}","comments_url":"https://api.github.com/repos/tyarkoni/transitions/issues/80/comments","events_url":"https://api.github.com/repos/tyarkoni/transitions/issues/80/events","html_url":"https://github.com/tyarkoni/transitions/pull/80","id":131542939,"number":80,"title":"[RFC] before_check callback","user":{"login":"TheMysteriousX","id":590630,"avatar_url":"https://avatars.githubusercontent.com/u/590630?v=3","gravatar_id":"","url":"https://api.github.com/users/TheMysteriousX","html_url":"https://github.com/TheMysteriousX","followers_url":"https://api.github.com/users/TheMysteriousX/followers","following_url":"https://api.github.com/users/TheMysteriousX/following{/other_user}","gists_url":"https://api.github.com/users/TheMysteriousX/gists{/gist_id}","starred_url":"https://api.github.com/users/TheMysteriousX/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TheMysteriousX/subscriptions","organizations_url":"https://api.github.com/users/TheMysteriousX/orgs","repos_url":"https://api.github.com/users/TheMysteriousX/repos","events_url":"https://api.github.com/users/TheMysteriousX/events{/privacy}","received_events_url":"https://api.github.com/users/TheMysteriousX/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2016-02-05T03:13:50Z","updated_at":"2016-02-10T17:39:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tyarkoni/transitions/pulls/80","html_url":"https://github.com/tyarkoni/transitions/pull/80","diff_url":"https://github.com/tyarkoni/transitions/pull/80.diff","patch_url":"https://github.com/tyarkoni/transitions/pull/80.patch"},"body":"There seemed to be something of a consensus forming in #40, so I've continued with the changes I've already made by renaming the 'action' callback to 'before_check'.\r\n\r\nI've also added some compatibility code, which is maybe a bit coarse but should work to keep older code working.\r\n\r\nReworking __getattr__ was a little annoying, but it didn't correctly handle underscores.\r\n\r\nI'm not 100% certain this is ready to be merged, I'm just soliciting feedback at the moment."},"comment":{"url":"https://api.github.com/repos/tyarkoni/transitions/issues/comments/182496644","html_url":"https://github.com/tyarkoni/transitions/pull/80#issuecomment-182496644","issue_url":"https://api.github.com/repos/tyarkoni/transitions/issues/80","id":182496644,"user":{"login":"tyarkoni","id":303932,"avatar_url":"https://avatars.githubusercontent.com/u/303932?v=3","gravatar_id":"","url":"https://api.github.com/users/tyarkoni","html_url":"https://github.com/tyarkoni","followers_url":"https://api.github.com/users/tyarkoni/followers","following_url":"https://api.github.com/users/tyarkoni/following{/other_user}","gists_url":"https://api.github.com/users/tyarkoni/gists{/gist_id}","starred_url":"https://api.github.com/users/tyarkoni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tyarkoni/subscriptions","organizations_url":"https://api.github.com/users/tyarkoni/orgs","repos_url":"https://api.github.com/users/tyarkoni/repos","events_url":"https://api.github.com/users/tyarkoni/events{/privacy}","received_events_url":"https://api.github.com/users/tyarkoni/received_events","type":"User","site_admin":false},"created_at":"2016-02-10T17:39:19Z","updated_at":"2016-02-10T17:39:19Z","body":"Oh, and forgot to mention--the patch looks great to me otherwise; thanks @TheMysteriousX!"}}