{"action":"created","issue":{"url":"https://api.github.com/repos/purnam/purnam/issues/9","labels_url":"https://api.github.com/repos/purnam/purnam/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/purnam/purnam/issues/9/comments","events_url":"https://api.github.com/repos/purnam/purnam/issues/9/events","html_url":"https://github.com/purnam/purnam/pull/9","id":53653628,"number":9,"title":"find-arrow-positions bug","user":{"login":"natedev","id":886657,"avatar_url":"https://avatars.githubusercontent.com/u/886657?v=3","gravatar_id":"","url":"https://api.github.com/users/natedev","html_url":"https://github.com/natedev","followers_url":"https://api.github.com/users/natedev/followers","following_url":"https://api.github.com/users/natedev/following{/other_user}","gists_url":"https://api.github.com/users/natedev/gists{/gist_id}","starred_url":"https://api.github.com/users/natedev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/natedev/subscriptions","organizations_url":"https://api.github.com/users/natedev/orgs","repos_url":"https://api.github.com/users/natedev/repos","events_url":"https://api.github.com/users/natedev/events{/privacy}","received_events_url":"https://api.github.com/users/natedev/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-07T16:50:47Z","updated_at":"2015-01-07T22:48:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/purnam/purnam/pulls/9","html_url":"https://github.com/purnam/purnam/pull/9","diff_url":"https://github.com/purnam/purnam/pull/9.diff","patch_url":"https://github.com/purnam/purnam/pull/9.patch"},"body":"There is a bug in find-arrow-positions that occurs if there are multiple test statements in a fact/facts form and any side of the test is a symbol that would evaluate as being falsey (ex. false or nil).  find-arrow-positions uses an if statement to evaluate each symbol when parsing and so any falsey value will terminate the recur and thus return an invalid result for what positions all of the =>'s are in.\r\n\r\nI added a fix to recur on non-empty seq's to fix the issue as well as a test that should fail if this bug were to be reintroduced."},"comment":{"url":"https://api.github.com/repos/purnam/purnam/issues/comments/69106856","html_url":"https://github.com/purnam/purnam/pull/9#issuecomment-69106856","issue_url":"https://api.github.com/repos/purnam/purnam/issues/9","id":69106856,"user":{"login":"zcaudate","id":1455572,"avatar_url":"https://avatars.githubusercontent.com/u/1455572?v=3","gravatar_id":"","url":"https://api.github.com/users/zcaudate","html_url":"https://github.com/zcaudate","followers_url":"https://api.github.com/users/zcaudate/followers","following_url":"https://api.github.com/users/zcaudate/following{/other_user}","gists_url":"https://api.github.com/users/zcaudate/gists{/gist_id}","starred_url":"https://api.github.com/users/zcaudate/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zcaudate/subscriptions","organizations_url":"https://api.github.com/users/zcaudate/orgs","repos_url":"https://api.github.com/users/zcaudate/repos","events_url":"https://api.github.com/users/zcaudate/events{/privacy}","received_events_url":"https://api.github.com/users/zcaudate/received_events","type":"User","site_admin":false},"created_at":"2015-01-07T22:48:31Z","updated_at":"2015-01-07T22:48:31Z","body":"Thanks for the PR. It's much appreciated!"}}