{"action":"created","issue":{"url":"https://api.github.com/repos/iwburns/nullshield/issues/32","repository_url":"https://api.github.com/repos/iwburns/nullshield","labels_url":"https://api.github.com/repos/iwburns/nullshield/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/iwburns/nullshield/issues/32/comments","events_url":"https://api.github.com/repos/iwburns/nullshield/issues/32/events","html_url":"https://github.com/iwburns/nullshield/pull/32","id":296146427,"number":32,"title":"restructure code to use a base abstract class","user":{"login":"ryanguill","id":1149698,"avatar_url":"https://avatars1.githubusercontent.com/u/1149698?v=4","gravatar_id":"","url":"https://api.github.com/users/ryanguill","html_url":"https://github.com/ryanguill","followers_url":"https://api.github.com/users/ryanguill/followers","following_url":"https://api.github.com/users/ryanguill/following{/other_user}","gists_url":"https://api.github.com/users/ryanguill/gists{/gist_id}","starred_url":"https://api.github.com/users/ryanguill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ryanguill/subscriptions","organizations_url":"https://api.github.com/users/ryanguill/orgs","repos_url":"https://api.github.com/users/ryanguill/repos","events_url":"https://api.github.com/users/ryanguill/events{/privacy}","received_events_url":"https://api.github.com/users/ryanguill/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-02-11T00:14:56Z","updated_at":"2018-02-11T02:06:52Z","closed_at":null,"author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/iwburns/nullshield/pulls/32","html_url":"https://github.com/iwburns/nullshield/pull/32","diff_url":"https://github.com/iwburns/nullshield/pull/32.diff","patch_url":"https://github.com/iwburns/nullshield/pull/32.patch"},"body":" which should clean up the type signatures."},"comment":{"url":"https://api.github.com/repos/iwburns/nullshield/issues/comments/364712895","html_url":"https://github.com/iwburns/nullshield/pull/32#issuecomment-364712895","issue_url":"https://api.github.com/repos/iwburns/nullshield/issues/32","id":364712895,"user":{"login":"iwburns","id":6172641,"avatar_url":"https://avatars2.githubusercontent.com/u/6172641?v=4","gravatar_id":"","url":"https://api.github.com/users/iwburns","html_url":"https://github.com/iwburns","followers_url":"https://api.github.com/users/iwburns/followers","following_url":"https://api.github.com/users/iwburns/following{/other_user}","gists_url":"https://api.github.com/users/iwburns/gists{/gist_id}","starred_url":"https://api.github.com/users/iwburns/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iwburns/subscriptions","organizations_url":"https://api.github.com/users/iwburns/orgs","repos_url":"https://api.github.com/users/iwburns/repos","events_url":"https://api.github.com/users/iwburns/events{/privacy}","received_events_url":"https://api.github.com/users/iwburns/received_events","type":"User","site_admin":false},"created_at":"2018-02-11T02:06:52Z","updated_at":"2018-02-11T02:06:52Z","author_association":"OWNER","body":"On the whole this looks really good to me!\r\n\r\nI did leave a few comments in-line and I've got one more request here:\r\ndo you mind adding more tests for `OptionT.of` and `OptionT.none` making sure that they do and do not throw in the proper cases?\r\n\r\nOther than those few things, I'm basically ready to merge this."}}
{"actor":{"display_login":"iwburns"}}