{"action":"created","issue":{"url":"https://api.github.com/repos/Hopac/Hopac/issues/74","labels_url":"https://api.github.com/repos/Hopac/Hopac/issues/74/labels{/name}","comments_url":"https://api.github.com/repos/Hopac/Hopac/issues/74/comments","events_url":"https://api.github.com/repos/Hopac/Hopac/issues/74/events","html_url":"https://github.com/Hopac/Hopac/issues/74","id":100238425,"number":74,"title":"Job Try Finally not catching exceptions","user":{"login":"mavnn","id":309312,"avatar_url":"https://avatars.githubusercontent.com/u/309312?v=3","gravatar_id":"","url":"https://api.github.com/users/mavnn","html_url":"https://github.com/mavnn","followers_url":"https://api.github.com/users/mavnn/followers","following_url":"https://api.github.com/users/mavnn/following{/other_user}","gists_url":"https://api.github.com/users/mavnn/gists{/gist_id}","starred_url":"https://api.github.com/users/mavnn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mavnn/subscriptions","organizations_url":"https://api.github.com/users/mavnn/orgs","repos_url":"https://api.github.com/users/mavnn/repos","events_url":"https://api.github.com/users/mavnn/events{/privacy}","received_events_url":"https://api.github.com/users/mavnn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-08-11T06:36:19Z","updated_at":"2015-08-11T07:32:03Z","closed_at":null,"body":"I could be doing something wrong somewhere, but this test fails for me (the exception is not caught):\r\n\r\n    [<Test>]\r\n    let ``Job try finally works`` () =\r\n        let finallyFired = ref false\r\n        let act = job {\r\n            try\r\n                failwith \"I will always fail!\"   \r\n            finally\r\n                finallyFired := true            \r\n        }\r\n        act |> Job.Global.run |> ignore\r\n        Assert.True(!finallyFired)\r\n\r\nThe same test for ``try ... with`` works fine. I'm afraid I don't yet understand enough of how the Job.tryFinally methods are implemented to have a good idea for a fix."},"comment":{"url":"https://api.github.com/repos/Hopac/Hopac/issues/comments/129739903","html_url":"https://github.com/Hopac/Hopac/issues/74#issuecomment-129739903","issue_url":"https://api.github.com/repos/Hopac/Hopac/issues/74","id":129739903,"user":{"login":"mavnn","id":309312,"avatar_url":"https://avatars.githubusercontent.com/u/309312?v=3","gravatar_id":"","url":"https://api.github.com/users/mavnn","html_url":"https://github.com/mavnn","followers_url":"https://api.github.com/users/mavnn/followers","following_url":"https://api.github.com/users/mavnn/following{/other_user}","gists_url":"https://api.github.com/users/mavnn/gists{/gist_id}","starred_url":"https://api.github.com/users/mavnn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mavnn/subscriptions","organizations_url":"https://api.github.com/users/mavnn/orgs","repos_url":"https://api.github.com/users/mavnn/repos","events_url":"https://api.github.com/users/mavnn/events{/privacy}","received_events_url":"https://api.github.com/users/mavnn/received_events","type":"User","site_admin":false},"created_at":"2015-08-11T07:32:03Z","updated_at":"2015-08-11T07:32:03Z","body":"... doh. Now I feel stupid!"}}