{"action":"created","issue":{"url":"https://api.github.com/repos/nolanw/HTMLReader/issues/52","labels_url":"https://api.github.com/repos/nolanw/HTMLReader/issues/52/labels{/name}","comments_url":"https://api.github.com/repos/nolanw/HTMLReader/issues/52/comments","events_url":"https://api.github.com/repos/nolanw/HTMLReader/issues/52/events","html_url":"https://github.com/nolanw/HTMLReader/pull/52","id":121518808,"number":52,"title":"Added a comment explaining a potential test suite failure","user":{"login":"zoul","id":95329,"avatar_url":"https://avatars.githubusercontent.com/u/95329?v=3","gravatar_id":"","url":"https://api.github.com/users/zoul","html_url":"https://github.com/zoul","followers_url":"https://api.github.com/users/zoul/followers","following_url":"https://api.github.com/users/zoul/following{/other_user}","gists_url":"https://api.github.com/users/zoul/gists{/gist_id}","starred_url":"https://api.github.com/users/zoul/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zoul/subscriptions","organizations_url":"https://api.github.com/users/zoul/orgs","repos_url":"https://api.github.com/users/zoul/repos","events_url":"https://api.github.com/users/zoul/events{/privacy}","received_events_url":"https://api.github.com/users/zoul/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-12-10T15:59:47Z","updated_at":"2015-12-10T23:06:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nolanw/HTMLReader/pulls/52","html_url":"https://github.com/nolanw/HTMLReader/pull/52","diff_url":"https://github.com/nolanw/HTMLReader/pull/52.diff","patch_url":"https://github.com/nolanw/HTMLReader/pull/52.patch"},"body":"The `Tests/html5lib` submodule has to be checked out, otherwise an assert in the test suite fails. I have added an explanation above the assert to save people like me some thinking. (See #51.)"},"comment":{"url":"https://api.github.com/repos/nolanw/HTMLReader/issues/comments/163781571","html_url":"https://github.com/nolanw/HTMLReader/pull/52#issuecomment-163781571","issue_url":"https://api.github.com/repos/nolanw/HTMLReader/issues/52","id":163781571,"user":{"login":"nolanw","id":177228,"avatar_url":"https://avatars.githubusercontent.com/u/177228?v=3","gravatar_id":"","url":"https://api.github.com/users/nolanw","html_url":"https://github.com/nolanw","followers_url":"https://api.github.com/users/nolanw/followers","following_url":"https://api.github.com/users/nolanw/following{/other_user}","gists_url":"https://api.github.com/users/nolanw/gists{/gist_id}","starred_url":"https://api.github.com/users/nolanw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nolanw/subscriptions","organizations_url":"https://api.github.com/users/nolanw/orgs","repos_url":"https://api.github.com/users/nolanw/repos","events_url":"https://api.github.com/users/nolanw/events{/privacy}","received_events_url":"https://api.github.com/users/nolanw/received_events","type":"User","site_admin":false},"created_at":"2015-12-10T23:06:39Z","updated_at":"2015-12-10T23:06:39Z","body":"I think I'd rather take the assert out (or rework it) so tests run without getting submodules. And maybe mention in the README that one needs to clone submodules in order to run the full test suite. Does that sound good?"}}