{"action":"created","issue":{"url":"https://api.github.com/repos/libgit2/libgit2/issues/4099","repository_url":"https://api.github.com/repos/libgit2/libgit2","labels_url":"https://api.github.com/repos/libgit2/libgit2/issues/4099/labels{/name}","comments_url":"https://api.github.com/repos/libgit2/libgit2/issues/4099/comments","events_url":"https://api.github.com/repos/libgit2/libgit2/issues/4099/events","html_url":"https://github.com/libgit2/libgit2/issues/4099","id":204000125,"number":4099,"title":"revwalk produces incorrect results with GIT_SORT_TIME","user":{"login":"adamniedzielski","id":2426386,"avatar_url":"https://avatars.githubusercontent.com/u/2426386?v=3","gravatar_id":"","url":"https://api.github.com/users/adamniedzielski","html_url":"https://github.com/adamniedzielski","followers_url":"https://api.github.com/users/adamniedzielski/followers","following_url":"https://api.github.com/users/adamniedzielski/following{/other_user}","gists_url":"https://api.github.com/users/adamniedzielski/gists{/gist_id}","starred_url":"https://api.github.com/users/adamniedzielski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adamniedzielski/subscriptions","organizations_url":"https://api.github.com/users/adamniedzielski/orgs","repos_url":"https://api.github.com/users/adamniedzielski/repos","events_url":"https://api.github.com/users/adamniedzielski/events{/privacy}","received_events_url":"https://api.github.com/users/adamniedzielski/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-01-30T12:41:12Z","updated_at":"2017-02-17T13:41:30Z","closed_at":null,"body":"I reported this issue in `rugged` repo - https://github.com/libgit2/rugged/issues/673. It includes an example in Ruby which may be helpful. I believe that this issue is caused by `libgit2` and that it is a regression.\r\n\r\n### Reproduction steps\r\n\r\nGiven a repository:\r\n\r\n```\r\n* commit C (HEAD)\r\n|\r\n* commit B\r\n|\r\n* commit A\r\n```\r\n\r\nwalk from A, hiding C, using `GIT_SORT_TIME` order. See #4100 for a failing test case.\r\n\r\n### Expected behavior\r\n\r\n0 commits returned\r\n\r\n### Actual behavior\r\n\r\n1 commit returned (commit A).\r\n\r\n### Version of libgit2 (release number or SHA1)\r\n\r\ndad3c319d76f58bc600b53650723d0d3cc9e6feb\r\n\r\n### Operating system(s) tested\r\n\r\nmacOS 10.12.2\r\n"},"comment":{"url":"https://api.github.com/repos/libgit2/libgit2/issues/comments/280652168","html_url":"https://github.com/libgit2/libgit2/issues/4099#issuecomment-280652168","issue_url":"https://api.github.com/repos/libgit2/libgit2/issues/4099","id":280652168,"user":{"login":"carlosmn","id":335443,"avatar_url":"https://avatars.githubusercontent.com/u/335443?v=3","gravatar_id":"","url":"https://api.github.com/users/carlosmn","html_url":"https://github.com/carlosmn","followers_url":"https://api.github.com/users/carlosmn/followers","following_url":"https://api.github.com/users/carlosmn/following{/other_user}","gists_url":"https://api.github.com/users/carlosmn/gists{/gist_id}","starred_url":"https://api.github.com/users/carlosmn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/carlosmn/subscriptions","organizations_url":"https://api.github.com/users/carlosmn/orgs","repos_url":"https://api.github.com/users/carlosmn/repos","events_url":"https://api.github.com/users/carlosmn/events{/privacy}","received_events_url":"https://api.github.com/users/carlosmn/received_events","type":"User","site_admin":true},"created_at":"2017-02-17T13:41:30Z","updated_at":"2017-02-17T13:41:30Z","body":"Yes, it looks like we might still get uninteresting commits in the queue for the time iterator, so we do need the `if` to skip them."}}
{"actor":{"display_login":"carlosmn"}}