{"action":"created","issue":{"url":"https://api.github.com/repos/avocado-framework/avocado/issues/344","labels_url":"https://api.github.com/repos/avocado-framework/avocado/issues/344/labels{/name}","comments_url":"https://api.github.com/repos/avocado-framework/avocado/issues/344/comments","events_url":"https://api.github.com/repos/avocado-framework/avocado/issues/344/events","html_url":"https://github.com/avocado-framework/avocado/pull/344","id":55192871,"number":344,"title":"avocado: Add another branch in test detection heuristics","user":{"login":"lmr","id":296807,"avatar_url":"https://avatars.githubusercontent.com/u/296807?v=3","gravatar_id":"","url":"https://api.github.com/users/lmr","html_url":"https://github.com/lmr","followers_url":"https://api.github.com/users/lmr/followers","following_url":"https://api.github.com/users/lmr/following{/other_user}","gists_url":"https://api.github.com/users/lmr/gists{/gist_id}","starred_url":"https://api.github.com/users/lmr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lmr/subscriptions","organizations_url":"https://api.github.com/users/lmr/orgs","repos_url":"https://api.github.com/users/lmr/repos","events_url":"https://api.github.com/users/lmr/events{/privacy}","received_events_url":"https://api.github.com/users/lmr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-22T18:51:50Z","updated_at":"2015-01-23T00:39:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/avocado-framework/avocado/pulls/344","html_url":"https://github.com/avocado-framework/avocado/pull/344","diff_url":"https://github.com/avocado-framework/avocado/pull/344.diff","patch_url":"https://github.com/avocado-framework/avocado/pull/344.patch"},"body":"It occurred to us that we can tell apart a buggy avocado\r\ntest from a random buggy module since all avocado instrumented\r\ntests will have an avocado import. Therefore, let's do a regexp\r\nsearch on the module to see if we find an avocado import header.\r\n\r\nOf course, this still has room for mistakes, but this is a\r\nbest effort enterprise anyway. Update the diagram with the\r\nextra branch in the test resolution.\r\n\r\nSigned-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>"},"comment":{"url":"https://api.github.com/repos/avocado-framework/avocado/issues/comments/71128454","html_url":"https://github.com/avocado-framework/avocado/pull/344#issuecomment-71128454","issue_url":"https://api.github.com/repos/avocado-framework/avocado/issues/344","id":71128454,"user":{"login":"adereis","id":739481,"avatar_url":"https://avatars.githubusercontent.com/u/739481?v=3","gravatar_id":"","url":"https://api.github.com/users/adereis","html_url":"https://github.com/adereis","followers_url":"https://api.github.com/users/adereis/followers","following_url":"https://api.github.com/users/adereis/following{/other_user}","gists_url":"https://api.github.com/users/adereis/gists{/gist_id}","starred_url":"https://api.github.com/users/adereis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adereis/subscriptions","organizations_url":"https://api.github.com/users/adereis/orgs","repos_url":"https://api.github.com/users/adereis/repos","events_url":"https://api.github.com/users/adereis/events{/privacy}","received_events_url":"https://api.github.com/users/adereis/received_events","type":"User","site_admin":false},"created_at":"2015-01-23T00:39:31Z","updated_at":"2015-01-23T00:39:31Z","body":"LGTM (I was discussing this with lmr on IRC). But I won't merge it because I don't think my superficial review here on github is enough."}}