{"action":"created","comment":{"url":"https://api.github.com/repos/bronto/javasphinx/pulls/comments/95393930","pull_request_review_id":15942869,"id":95393930,"diff_hunk":"@@ -192,8 +200,14 @@ def generate_from_source_file(doc_compiler, source_file, cache_dir):\n     except Exception:\n         util.unexpected('Unexpected exception while parsing %s', source_file)\n \n+    documents = {}\n     try:\n-        documents = doc_compiler.compile(ast)\n+        if source_file.endswith(\"package-info.java\"):\n+            for _, node in ast.filter(javalang.tree.PackageDeclaration):","path":"javasphinx/apidoc.py","position":75,"original_position":75,"commit_id":"2119bca965dc05e27e4c4d7cb161b850e8011b41","original_commit_id":"2119bca965dc05e27e4c4d7cb161b850e8011b41","user":{"login":"c2nes","id":1040778,"avatar_url":"https://avatars.githubusercontent.com/u/1040778?v=3","gravatar_id":"","url":"https://api.github.com/users/c2nes","html_url":"https://github.com/c2nes","followers_url":"https://api.github.com/users/c2nes/followers","following_url":"https://api.github.com/users/c2nes/following{/other_user}","gists_url":"https://api.github.com/users/c2nes/gists{/gist_id}","starred_url":"https://api.github.com/users/c2nes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/c2nes/subscriptions","organizations_url":"https://api.github.com/users/c2nes/orgs","repos_url":"https://api.github.com/users/c2nes/repos","events_url":"https://api.github.com/users/c2nes/events{/privacy}","received_events_url":"https://api.github.com/users/c2nes/received_events","type":"User","site_admin":false},"body":"I think a filter here is overkill. You should just be able to check that `ast.package` is not `None`.","created_at":"2017-01-10T15:58:09Z","updated_at":"2017-01-10T16:01:21Z","html_url":"https://github.com/bronto/javasphinx/pull/57#discussion_r95393930","pull_request_url":"https://api.github.com/repos/bronto/javasphinx/pulls/57","_links":{"self":{"href":"https://api.github.com/repos/bronto/javasphinx/pulls/comments/95393930"},"html":{"href":"https://github.com/bronto/javasphinx/pull/57#discussion_r95393930"},"pull_request":{"href":"https://api.github.com/repos/bronto/javasphinx/pulls/57"}}},"pull_request":{"url":"https://api.github.com/repos/bronto/javasphinx/pulls/57","id":100861950,"html_url":"https://github.com/bronto/javasphinx/pull/57","diff_url":"https://github.com/bronto/javasphinx/pull/57.diff","patch_url":"https://github.com/bronto/javasphinx/pull/57.patch","issue_url":"https://api.github.com/repos/bronto/javasphinx/issues/57","number":57,"state":"open","locked":false,"title":"Closes #12","user":{"login":"manoelcampos","id":261605,"avatar_url":"https://avatars.githubusercontent.com/u/261605?v=3","gravatar_id":"","url":"https://api.github.com/users/manoelcampos","html_url":"https://github.com/manoelcampos","followers_url":"https://api.github.com/users/manoelcampos/followers","following_url":"https://api.github.com/users/manoelcampos/following{/other_user}","gists_url":"https://api.github.com/users/manoelcampos/gists{/gist_id}","starred_url":"https://api.github.com/users/manoelcampos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/manoelcampos/subscriptions","organizations_url":"https://api.github.com/users/manoelcampos/orgs","repos_url":"https://api.github.com/users/manoelcampos/repos","events_url":"https://api.github.com/users/manoelcampos/events{/privacy}","received_events_url":"https://api.github.com/users/manoelcampos/received_events","type":"User","site_admin":false},"body":"Added support to read package-info.java files and load package documentation\r\nto insert into the generated package-index files.\r\n\r\nNow the list of packages returned by the method generate_documents\r\nfrom apidoc.py returns a dictionary instead of list containing just\r\nthe package names.\r\n\r\nThat dictionary uses the package name as a key\r\nand each value is a a tuple in the same format used\r\nfor the documents dict returned by that same method.\r\n\r\nSuch a tuple contains 3 values: (package_name, class_name, parsed_javadoc).\r\nFor the elements of the packages dict, the only difference is that\r\ninstead of the class_name, it is repeated the package name.","created_at":"2017-01-10T12:12:36Z","updated_at":"2017-01-10T16:01:21Z","closed_at":null,"merged_at":null,"merge_commit_sha":"eea73d7bfb7560a0c728436e3b9c69e480983aba","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/bronto/javasphinx/pulls/57/commits","review_comments_url":"https://api.github.com/repos/bronto/javasphinx/pulls/57/comments","review_comment_url":"https://api.github.com/repos/bronto/javasphinx/pulls/comments{/number}","comments_url":"https://api.github.com/repos/bronto/javasphinx/issues/57/comments","statuses_url":"https://api.github.com/repos/bronto/javasphinx/statuses/2119bca965dc05e27e4c4d7cb161b850e8011b41","head":{"label":"manoelcampos:master","ref":"master","sha":"2119bca965dc05e27e4c4d7cb161b850e8011b41","user":{"login":"manoelcampos","id":261605,"avatar_url":"https://avatars.githubusercontent.com/u/261605?v=3","gravatar_id":"","url":"https://api.github.com/users/manoelcampos","html_url":"https://github.com/manoelcampos","followers_url":"https://api.github.com/users/manoelcampos/followers","following_url":"https://api.github.com/users/manoelcampos/following{/other_user}","gists_url":"https://api.github.com/users/manoelcampos/gists{/gist_id}","starred_url":"https://api.github.com/users/manoelcampos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/manoelcampos/subscriptions","organizations_url":"https://api.github.com/users/manoelcampos/orgs","repos_url":"https://api.github.com/users/manoelcampos/repos","events_url":"https://api.github.com/users/manoelcampos/events{/privacy}","received_events_url":"https://api.github.com/users/manoelcampos/received_events","type":"User","site_admin":false},"repo":{"id":78348301,"name":"javasphinx","full_name":"manoelcampos/javasphinx","owner":{"login":"manoelcampos","id":261605,"avatar_url":"https://avatars.githubusercontent.com/u/261605?v=3","gravatar_id":"","url":"https://api.github.com/users/manoelcampos","html_url":"https://github.com/manoelcampos","followers_url":"https://api.github.com/users/manoelcampos/followers","following_url":"https://api.github.com/users/manoelcampos/following{/other_user}","gists_url":"https://api.github.com/users/manoelcampos/gists{/gist_id}","starred_url":"https://api.github.com/users/manoelcampos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/manoelcampos/subscriptions","organizations_url":"https://api.github.com/users/manoelcampos/orgs","repos_url":"https://api.github.com/users/manoelcampos/repos","events_url":"https://api.github.com/users/manoelcampos/events{/privacy}","received_events_url":"https://api.github.com/users/manoelcampos/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/manoelcampos/javasphinx","description":"Sphinx extension for documenting Java projects","fork":true,"url":"https://api.github.com/repos/manoelcampos/javasphinx","forks_url":"https://api.github.com/repos/manoelcampos/javasphinx/forks","keys_url":"https://api.github.com/repos/manoelcampos/javasphinx/keys{/key_id}","collaborators_url":"https://api.github.com/repos/manoelcampos/javasphinx/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/manoelcampos/javasphinx/teams","hooks_url":"https://api.github.com/repos/manoelcampos/javasphinx/hooks","issue_events_url":"https://api.github.com/repos/manoelcampos/javasphinx/issues/events{/number}","events_url":"https://api.github.com/repos/manoelcampos/javasphinx/events","assignees_url":"https://api.github.com/repos/manoelcampos/javasphinx/assignees{/user}","branches_url":"https://api.github.com/repos/manoelcampos/javasphinx/branches{/branch}","tags_url":"https://api.github.com/repos/manoelcampos/javasphinx/tags","blobs_url":"https://api.github.com/repos/manoelcampos/javasphinx/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/manoelcampos/javasphinx/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/manoelcampos/javasphinx/git/refs{/sha}","trees_url":"https://api.github.com/repos/manoelcampos/javasphinx/git/trees{/sha}","statuses_url":"https://api.github.com/repos/manoelcampos/javasphinx/statuses/{sha}","languages_url":"https://api.github.com/repos/manoelcampos/javasphinx/languages","stargazers_url":"https://api.github.com/repos/manoelcampos/javasphinx/stargazers","contributors_url":"https://api.github.com/repos/manoelcampos/javasphinx/contributors","subscribers_url":"https://api.github.com/repos/manoelcampos/javasphinx/subscribers","subscription_url":"https://api.github.com/repos/manoelcampos/javasphinx/subscription","commits_url":"https://api.github.com/repos/manoelcampos/javasphinx/commits{/sha}","git_commits_url":"https://api.github.com/repos/manoelcampos/javasphinx/git/commits{/sha}","comments_url":"https://api.github.com/repos/manoelcampos/javasphinx/comments{/number}","issue_comment_url":"https://api.github.com/repos/manoelcampos/javasphinx/issues/comments{/number}","contents_url":"https://api.github.com/repos/manoelcampos/javasphinx/contents/{+path}","compare_url":"https://api.github.com/repos/manoelcampos/javasphinx/compare/{base}...{head}","merges_url":"https://api.github.com/repos/manoelcampos/javasphinx/merges","archive_url":"https://api.github.com/repos/manoelcampos/javasphinx/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/manoelcampos/javasphinx/downloads","issues_url":"https://api.github.com/repos/manoelcampos/javasphinx/issues{/number}","pulls_url":"https://api.github.com/repos/manoelcampos/javasphinx/pulls{/number}","milestones_url":"https://api.github.com/repos/manoelcampos/javasphinx/milestones{/number}","notifications_url":"https://api.github.com/repos/manoelcampos/javasphinx/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/manoelcampos/javasphinx/labels{/name}","releases_url":"https://api.github.com/repos/manoelcampos/javasphinx/releases{/id}","deployments_url":"https://api.github.com/repos/manoelcampos/javasphinx/deployments","created_at":"2017-01-08T14:33:51Z","updated_at":"2017-01-08T14:33:56Z","pushed_at":"2017-01-10T12:12:03Z","git_url":"git://github.com/manoelcampos/javasphinx.git","ssh_url":"git@github.com:manoelcampos/javasphinx.git","clone_url":"https://github.com/manoelcampos/javasphinx.git","svn_url":"https://github.com/manoelcampos/javasphinx","homepage":"","size":165,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"bronto:master","ref":"master","sha":"c2edf5a1cf45e8a68e9c7362dce47ac6fef71b9a","user":{"login":"bronto","id":1382287,"avatar_url":"https://avatars.githubusercontent.com/u/1382287?v=3","gravatar_id":"","url":"https://api.github.com/users/bronto","html_url":"https://github.com/bronto","followers_url":"https://api.github.com/users/bronto/followers","following_url":"https://api.github.com/users/bronto/following{/other_user}","gists_url":"https://api.github.com/users/bronto/gists{/gist_id}","starred_url":"https://api.github.com/users/bronto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bronto/subscriptions","organizations_url":"https://api.github.com/users/bronto/orgs","repos_url":"https://api.github.com/users/bronto/repos","events_url":"https://api.github.com/users/bronto/events{/privacy}","received_events_url":"https://api.github.com/users/bronto/received_events","type":"Organization","site_admin":false},"repo":{"id":6677881,"name":"javasphinx","full_name":"bronto/javasphinx","owner":{"login":"bronto","id":1382287,"avatar_url":"https://avatars.githubusercontent.com/u/1382287?v=3","gravatar_id":"","url":"https://api.github.com/users/bronto","html_url":"https://github.com/bronto","followers_url":"https://api.github.com/users/bronto/followers","following_url":"https://api.github.com/users/bronto/following{/other_user}","gists_url":"https://api.github.com/users/bronto/gists{/gist_id}","starred_url":"https://api.github.com/users/bronto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bronto/subscriptions","organizations_url":"https://api.github.com/users/bronto/orgs","repos_url":"https://api.github.com/users/bronto/repos","events_url":"https://api.github.com/users/bronto/events{/privacy}","received_events_url":"https://api.github.com/users/bronto/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/bronto/javasphinx","description":"Sphinx extension for documenting Java projects","fork":false,"url":"https://api.github.com/repos/bronto/javasphinx","forks_url":"https://api.github.com/repos/bronto/javasphinx/forks","keys_url":"https://api.github.com/repos/bronto/javasphinx/keys{/key_id}","collaborators_url":"https://api.github.com/repos/bronto/javasphinx/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/bronto/javasphinx/teams","hooks_url":"https://api.github.com/repos/bronto/javasphinx/hooks","issue_events_url":"https://api.github.com/repos/bronto/javasphinx/issues/events{/number}","events_url":"https://api.github.com/repos/bronto/javasphinx/events","assignees_url":"https://api.github.com/repos/bronto/javasphinx/assignees{/user}","branches_url":"https://api.github.com/repos/bronto/javasphinx/branches{/branch}","tags_url":"https://api.github.com/repos/bronto/javasphinx/tags","blobs_url":"https://api.github.com/repos/bronto/javasphinx/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/bronto/javasphinx/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/bronto/javasphinx/git/refs{/sha}","trees_url":"https://api.github.com/repos/bronto/javasphinx/git/trees{/sha}","statuses_url":"https://api.github.com/repos/bronto/javasphinx/statuses/{sha}","languages_url":"https://api.github.com/repos/bronto/javasphinx/languages","stargazers_url":"https://api.github.com/repos/bronto/javasphinx/stargazers","contributors_url":"https://api.github.com/repos/bronto/javasphinx/contributors","subscribers_url":"https://api.github.com/repos/bronto/javasphinx/subscribers","subscription_url":"https://api.github.com/repos/bronto/javasphinx/subscription","commits_url":"https://api.github.com/repos/bronto/javasphinx/commits{/sha}","git_commits_url":"https://api.github.com/repos/bronto/javasphinx/git/commits{/sha}","comments_url":"https://api.github.com/repos/bronto/javasphinx/comments{/number}","issue_comment_url":"https://api.github.com/repos/bronto/javasphinx/issues/comments{/number}","contents_url":"https://api.github.com/repos/bronto/javasphinx/contents/{+path}","compare_url":"https://api.github.com/repos/bronto/javasphinx/compare/{base}...{head}","merges_url":"https://api.github.com/repos/bronto/javasphinx/merges","archive_url":"https://api.github.com/repos/bronto/javasphinx/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/bronto/javasphinx/downloads","issues_url":"https://api.github.com/repos/bronto/javasphinx/issues{/number}","pulls_url":"https://api.github.com/repos/bronto/javasphinx/pulls{/number}","milestones_url":"https://api.github.com/repos/bronto/javasphinx/milestones{/number}","notifications_url":"https://api.github.com/repos/bronto/javasphinx/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/bronto/javasphinx/labels{/name}","releases_url":"https://api.github.com/repos/bronto/javasphinx/releases{/id}","deployments_url":"https://api.github.com/repos/bronto/javasphinx/deployments","created_at":"2012-11-13T21:22:53Z","updated_at":"2017-01-08T09:32:28Z","pushed_at":"2017-01-10T12:12:37Z","git_url":"git://github.com/bronto/javasphinx.git","ssh_url":"git@github.com:bronto/javasphinx.git","clone_url":"https://github.com/bronto/javasphinx.git","svn_url":"https://github.com/bronto/javasphinx","homepage":"","size":196,"stargazers_count":52,"watchers_count":52,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":30,"mirror_url":null,"open_issues_count":16,"forks":30,"open_issues":16,"watchers":52,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/bronto/javasphinx/pulls/57"},"html":{"href":"https://github.com/bronto/javasphinx/pull/57"},"issue":{"href":"https://api.github.com/repos/bronto/javasphinx/issues/57"},"comments":{"href":"https://api.github.com/repos/bronto/javasphinx/issues/57/comments"},"review_comments":{"href":"https://api.github.com/repos/bronto/javasphinx/pulls/57/comments"},"review_comment":{"href":"https://api.github.com/repos/bronto/javasphinx/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/bronto/javasphinx/pulls/57/commits"},"statuses":{"href":"https://api.github.com/repos/bronto/javasphinx/statuses/2119bca965dc05e27e4c4d7cb161b850e8011b41"}}}}
{"actor":{"display_login":"c2nes"}}