{"action":"created","issue":{"url":"https://api.github.com/repos/purescript/purescript-maps/issues/43","labels_url":"https://api.github.com/repos/purescript/purescript-maps/issues/43/labels{/name}","comments_url":"https://api.github.com/repos/purescript/purescript-maps/issues/43/comments","events_url":"https://api.github.com/repos/purescript/purescript-maps/issues/43/events","html_url":"https://github.com/purescript/purescript-maps/pull/43","id":107297624,"number":43,"title":"fix peek to return Maybe, update docs","user":{"login":"jbrownson","id":808113,"avatar_url":"https://avatars.githubusercontent.com/u/808113?v=3","gravatar_id":"","url":"https://api.github.com/users/jbrownson","html_url":"https://github.com/jbrownson","followers_url":"https://api.github.com/users/jbrownson/followers","following_url":"https://api.github.com/users/jbrownson/following{/other_user}","gists_url":"https://api.github.com/users/jbrownson/gists{/gist_id}","starred_url":"https://api.github.com/users/jbrownson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbrownson/subscriptions","organizations_url":"https://api.github.com/users/jbrownson/orgs","repos_url":"https://api.github.com/users/jbrownson/repos","events_url":"https://api.github.com/users/jbrownson/events{/privacy}","received_events_url":"https://api.github.com/users/jbrownson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-09-19T00:02:26Z","updated_at":"2015-09-19T00:28:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/purescript/purescript-maps/pulls/43","html_url":"https://github.com/purescript/purescript-maps/pull/43","diff_url":"https://github.com/purescript/purescript-maps/pull/43.diff","patch_url":"https://github.com/purescript/purescript-maps/pull/43.patch"},"body":"After agreement in IRC w/ paf31, peek should really return something in Maybe otherwise `(SM.new >>= flip SM.peek \"asdf\") :: forall s. Eff (st :: ST s) String` returns `undefined`.\r\n\r\nIt looks like the docs for unions was also out of date, that change in the docs is unrelated to my code changes here.\r\n\r\nPer discussion in IRC I followed the style of purescript-arrays' peek function in the implementation."},"comment":{"url":"https://api.github.com/repos/purescript/purescript-maps/issues/comments/141600516","html_url":"https://github.com/purescript/purescript-maps/pull/43#issuecomment-141600516","issue_url":"https://api.github.com/repos/purescript/purescript-maps/issues/43","id":141600516,"user":{"login":"paf31","id":630306,"avatar_url":"https://avatars.githubusercontent.com/u/630306?v=3","gravatar_id":"","url":"https://api.github.com/users/paf31","html_url":"https://github.com/paf31","followers_url":"https://api.github.com/users/paf31/followers","following_url":"https://api.github.com/users/paf31/following{/other_user}","gists_url":"https://api.github.com/users/paf31/gists{/gist_id}","starred_url":"https://api.github.com/users/paf31/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paf31/subscriptions","organizations_url":"https://api.github.com/users/paf31/orgs","repos_url":"https://api.github.com/users/paf31/repos","events_url":"https://api.github.com/users/paf31/events{/privacy}","received_events_url":"https://api.github.com/users/paf31/received_events","type":"User","site_admin":false},"created_at":"2015-09-19T00:28:20Z","updated_at":"2015-09-19T00:28:20Z","body":"Possibly, but it would be possible to create the same thing for `undefined`.\r\n\r\nThis looks good to merge to me. @michaelficarra ?"}}