GitHub PullRequestEvent at 2014-11-24 10:04:48
Repository: apel
Actor: tofu-rocketry
Payload: {"action":"closed","number":50,"pull_request":{"url":"https://api.github.com/repos/apel/apel/pulls/50","id":23971277,"html_url":"https://github.com/apel/apel/pull/50","diff_url":"https://github.com/apel/apel/pull/50.diff","patch_url":"https://github.com/apel/apel/pull/50.patch","issue_url":"https://api.github.com/repos/apel/apel/issues/50","number":50,"state":"closed","locked":false,"title":"Improve code health","user":{"login":"tofu-rocketry","id":4836233,"avatar_url":"https://avatars.githubusercontent.com/u/4836233?v=3","gravatar_id":"","url":"https://api.github.com/users/tofu-rocketry","html_url":"https://github.com/tofu-rocketry","followers_url":"https://api.github.com/users/tofu-rocketry/followers","following_url":"https://api.github.com/users/tofu-rocketry/following{/other_user}","gists_url":"https://api.github.com/users/tofu-rocketry/gists{/gist_id}","starred_url":"https://api.github.com/users/tofu-rocketry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tofu-rocketry/subscriptions","organizations_url":"https://api.github.com/users/tofu-rocketry/orgs","repos_url":"https://api.github.com/users/tofu-rocketry/repos","events_url":"https://api.github.com/users/tofu-rocketry/events{/privacy}","received_events_url":"https://api.github.com/users/tofu-rocketry/received_events","type":"User","site_admin":false},"body":"Lots of little changes that shouldn't alter the way the code works (though the change to the logging commands might reduce overhead for messages that aren't logged).\r\n\r\n- Change triple quoted comments to plain comments.\r\n- Change string formatting to use arguments instead in the logging commands.\r\n- Unpack tuples when passed to logging functions as they should just be a\r\n  series of arguments.\r\n- Comment out one unused import.\r\n- Remove other unused imports.\r\n- Rename format variable as it clashes with built-in.\r\n- Tidy some white space.\r\n- Correct `not x is y` tests to `x is not y`.\r\n- Sort out some ambiguous indentation.\r\n- Compact some long lines.\r\n- Correct comparisons to None from `== None` to `is None`","created_at":"2014-11-06T10:29:44Z","updated_at":"2014-11-24T10:04:48Z","closed_at":"2014-11-24T10:04:48Z","merged_at":"2014-11-24T10:04:48Z","merge_commit_sha":"521927db12e5977e21921ea8819b5b5066645fea","assignee":{"login":"stuartpullinger","id":2781427,"avatar_url":"https://avatars.githubusercontent.com/u/2781427?v=3","gravatar_id":"","url":"https://api.github.com/users/stuartpullinger","html_url":"https://github.com/stuartpullinger","followers_url":"https://api.github.com/users/stuartpullinger/followers","following_url":"https://api.github.com/users/stuartpullinger/following{/other_user}","gists_url":"https://api.github.com/users/stuartpullinger/gists{/gist_id}","starred_url":"https://api.github.com/users/stuartpullinger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stuartpullinger/subscriptions","organizations_url":"https://api.github.com/users/stuartpullinger/orgs","repos_url":"https://api.github.com/users/stuartpullinger/repos","events_url":"https://api.github.com/users/stuartpullinger/events{/privacy}","received_events_url":"https://api.github.com/users/stuartpullinger/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/apel/apel/milestones/6","labels_url":"https://api.github.com/repos/apel/apel/milestones/6/labels","id":818118,"number":6,"title":"1.4.0","description":"","creator":{"login":"tofu-rocketry","id":4836233,"avatar_url":"https://avatars.githubusercontent.com/u/4836233?v=3","gravatar_id":"","url":"https://api.github.com/users/tofu-rocketry","html_url":"https://github.com/tofu-rocketry","followers_url":"https://api.github.com/users/tofu-rocketry/followers","following_url":"https://api.github.com/users/tofu-rocketry/following{/other_user}","gists_url":"https://api.github.com/users/tofu-rocketry/gists{/gist_id}","starred_url":"https://api.github.com/users/tofu-rocketry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tofu-rocketry/subscriptions","organizations_url":"https://api.github.com/users/tofu-rocketry/orgs","repos_url":"https://api.github.com/users/tofu-rocketry/repos","events_url":"https://api.github.com/users/tofu-rocketry/events{/privacy}","received_events_url":"https://api.github.com/users/tofu-rocketry/received_events","type":"User","site_admin":false},"open_issues":6,"closed_issues":6,"state":"open","created_at":"2014-10-08T09:45:19Z","updated_at":"2014-11-24T10:04:48Z","due_on":"2014-12-17T00:00:00Z","closed_at":null},"commits_url":"https://api.github.com/repos/apel/apel/pulls/50/commits","review_comments_url":"https://api.github.com/repos/apel/apel/pulls/50/comments","review_comment_url":"https://api.github.com/repos/apel/apel/pulls/comments/{number}","comments_url":"https://api.github.com/repos/apel/apel/issues/50/comments","statuses_url":"https://api.github.com/repos/apel/apel/statuses/eb154f96fd539067033e99292018478744385726","head":{"label":"apel:code_health","ref":"code_health","sha":"eb154f96fd539067033e99292018478744385726","user":{"login":"apel","id":3273995,"avatar_url":"https://avatars.githubusercontent.com/u/3273995?v=3","gravatar_id":"","url":"https://api.github.com/users/apel","html_url":"https://github.com/apel","followers_url":"https://api.github.com/users/apel/followers","following_url":"https://api.github.com/users/apel/following{/other_user}","gists_url":"https://api.github.com/users/apel/gists{/gist_id}","starred_url":"https://api.github.com/users/apel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apel/subscriptions","organizations_url":"https://api.github.com/users/apel/orgs","repos_url":"https://api.github.com/users/apel/repos","events_url":"https://api.github.com/users/apel/events{/privacy}","received_events_url":"https://api.github.com/users/apel/received_events","type":"Organization","site_admin":false},"repo":{"id":6702593,"name":"apel","full_name":"apel/apel","owner":{"login":"apel","id":3273995,"avatar_url":"https://avatars.githubusercontent.com/u/3273995?v=3","gravatar_id":"","url":"https://api.github.com/users/apel","html_url":"https://github.com/apel","followers_url":"https://api.github.com/users/apel/followers","following_url":"https://api.github.com/users/apel/following{/other_user}","gists_url":"https://api.github.com/users/apel/gists{/gist_id}","starred_url":"https://api.github.com/users/apel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apel/subscriptions","organizations_url":"https://api.github.com/users/apel/orgs","repos_url":"https://api.github.com/users/apel/repos","events_url":"https://api.github.com/users/apel/events{/privacy}","received_events_url":"https://api.github.com/users/apel/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/apel/apel","description":"APEL client and server source code.","fork":false,"url":"https://api.github.com/repos/apel/apel","forks_url":"https://api.github.com/repos/apel/apel/forks","keys_url":"https://api.github.com/repos/apel/apel/keys{/key_id}","collaborators_url":"https://api.github.com/repos/apel/apel/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/apel/apel/teams","hooks_url":"https://api.github.com/repos/apel/apel/hooks","issue_events_url":"https://api.github.com/repos/apel/apel/issues/events{/number}","events_url":"https://api.github.com/repos/apel/apel/events","assignees_url":"https://api.github.com/repos/apel/apel/assignees{/user}","branches_url":"https://api.github.com/repos/apel/apel/branches{/branch}","tags_url":"https://api.github.com/repos/apel/apel/tags","blobs_url":"https://api.github.com/repos/apel/apel/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/apel/apel/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/apel/apel/git/refs{/sha}","trees_url":"https://api.github.com/repos/apel/apel/git/trees{/sha}","statuses_url":"https://api.github.com/repos/apel/apel/statuses/{sha}","languages_url":"https://api.github.com/repos/apel/apel/languages","stargazers_url":"https://api.github.com/repos/apel/apel/stargazers","contributors_url":"https://api.github.com/repos/apel/apel/contributors","subscribers_url":"https://api.github.com/repos/apel/apel/subscribers","subscription_url":"https://api.github.com/repos/apel/apel/subscription","commits_url":"https://api.github.com/repos/apel/apel/commits{/sha}","git_commits_url":"https://api.github.com/repos/apel/apel/git/commits{/sha}","comments_url":"https://api.github.com/repos/apel/apel/comments{/number}","issue_comment_url":"https://api.github.com/repos/apel/apel/issues/comments/{number}","contents_url":"https://api.github.com/repos/apel/apel/contents/{+path}","compare_url":"https://api.github.com/repos/apel/apel/compare/{base}...{head}","merges_url":"https://api.github.com/repos/apel/apel/merges","archive_url":"https://api.github.com/repos/apel/apel/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/apel/apel/downloads","issues_url":"https://api.github.com/repos/apel/apel/issues{/number}","pulls_url":"https://api.github.com/repos/apel/apel/pulls{/number}","milestones_url":"https://api.github.com/repos/apel/apel/milestones{/number}","notifications_url":"https://api.github.com/repos/apel/apel/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/apel/apel/labels{/name}","releases_url":"https://api.github.com/repos/apel/apel/releases{/id}","created_at":"2012-11-15T09:52:44Z","updated_at":"2014-11-21T16:03:27Z","pushed_at":"2014-11-24T10:04:48Z","git_url":"git://github.com/apel/apel.git","ssh_url":"git@github.com:apel/apel.git","clone_url":"https://github.com/apel/apel.git","svn_url":"https://github.com/apel/apel","homepage":"apel.github.io","size":2945,"stargazers_count":3,"watchers_count":3,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":7,"mirror_url":null,"open_issues_count":11,"forks":7,"open_issues":11,"watchers":3,"default_branch":"dev"}},"base":{"label":"apel:dev","ref":"dev","sha":"5c8b04789a56ad2bfaa7de697a2c8d9af87e3f49","user":{"login":"apel","id":3273995,"avatar_url":"https://avatars.githubusercontent.com/u/3273995?v=3","gravatar_id":"","url":"https://api.github.com/users/apel","html_url":"https://github.com/apel","followers_url":"https://api.github.com/users/apel/followers","following_url":"https://api.github.com/users/apel/following{/other_user}","gists_url":"https://api.github.com/users/apel/gists{/gist_id}","starred_url":"https://api.github.com/users/apel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apel/subscriptions","organizations_url":"https://api.github.com/users/apel/orgs","repos_url":"https://api.github.com/users/apel/repos","events_url":"https://api.github.com/users/apel/events{/privacy}","received_events_url":"https://api.github.com/users/apel/received_events","type":"Organization","site_admin":false},"repo":{"id":6702593,"name":"apel","full_name":"apel/apel","owner":{"login":"apel","id":3273995,"avatar_url":"https://avatars.githubusercontent.com/u/3273995?v=3","gravatar_id":"","url":"https://api.github.com/users/apel","html_url":"https://github.com/apel","followers_url":"https://api.github.com/users/apel/followers","following_url":"https://api.github.com/users/apel/following{/other_user}","gists_url":"https://api.github.com/users/apel/gists{/gist_id}","starred_url":"https://api.github.com/users/apel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apel/subscriptions","organizations_url":"https://api.github.com/users/apel/orgs","repos_url":"https://api.github.com/users/apel/repos","events_url":"https://api.github.com/users/apel/events{/privacy}","received_events_url":"https://api.github.com/users/apel/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/apel/apel","description":"APEL client and server source code.","fork":false,"url":"https://api.github.com/repos/apel/apel","forks_url":"https://api.github.com/repos/apel/apel/forks","keys_url":"https://api.github.com/repos/apel/apel/keys{/key_id}","collaborators_url":"https://api.github.com/repos/apel/apel/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/apel/apel/teams","hooks_url":"https://api.github.com/repos/apel/apel/hooks","issue_events_url":"https://api.github.com/repos/apel/apel/issues/events{/number}","events_url":"https://api.github.com/repos/apel/apel/events","assignees_url":"https://api.github.com/repos/apel/apel/assignees{/user}","branches_url":"https://api.github.com/repos/apel/apel/branches{/branch}","tags_url":"https://api.github.com/repos/apel/apel/tags","blobs_url":"https://api.github.com/repos/apel/apel/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/apel/apel/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/apel/apel/git/refs{/sha}","trees_url":"https://api.github.com/repos/apel/apel/git/trees{/sha}","statuses_url":"https://api.github.com/repos/apel/apel/statuses/{sha}","languages_url":"https://api.github.com/repos/apel/apel/languages","stargazers_url":"https://api.github.com/repos/apel/apel/stargazers","contributors_url":"https://api.github.com/repos/apel/apel/contributors","subscribers_url":"https://api.github.com/repos/apel/apel/subscribers","subscription_url":"https://api.github.com/repos/apel/apel/subscription","commits_url":"https://api.github.com/repos/apel/apel/commits{/sha}","git_commits_url":"https://api.github.com/repos/apel/apel/git/commits{/sha}","comments_url":"https://api.github.com/repos/apel/apel/comments{/number}","issue_comment_url":"https://api.github.com/repos/apel/apel/issues/comments/{number}","contents_url":"https://api.github.com/repos/apel/apel/contents/{+path}","compare_url":"https://api.github.com/repos/apel/apel/compare/{base}...{head}","merges_url":"https://api.github.com/repos/apel/apel/merges","archive_url":"https://api.github.com/repos/apel/apel/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/apel/apel/downloads","issues_url":"https://api.github.com/repos/apel/apel/issues{/number}","pulls_url":"https://api.github.com/repos/apel/apel/pulls{/number}","milestones_url":"https://api.github.com/repos/apel/apel/milestones{/number}","notifications_url":"https://api.github.com/repos/apel/apel/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/apel/apel/labels{/name}","releases_url":"https://api.github.com/repos/apel/apel/releases{/id}","created_at":"2012-11-15T09:52:44Z","updated_at":"2014-11-21T16:03:27Z","pushed_at":"2014-11-24T10:04:48Z","git_url":"git://github.com/apel/apel.git","ssh_url":"git@github.com:apel/apel.git","clone_url":"https://github.com/apel/apel.git","svn_url":"https://github.com/apel/apel","homepage":"apel.github.io","size":2945,"stargazers_count":3,"watchers_count":3,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":7,"mirror_url":null,"open_issues_count":11,"forks":7,"open_issues":11,"watchers":3,"default_branch":"dev"}},"_links":{"self":{"href":"https://api.github.com/repos/apel/apel/pulls/50"},"html":{"href":"https://github.com/apel/apel/pull/50"},"issue":{"href":"https://api.github.com/repos/apel/apel/issues/50"},"comments":{"href":"https://api.github.com/repos/apel/apel/issues/50/comments"},"review_comments":{"href":"https://api.github.com/repos/apel/apel/pulls/50/comments"},"review_comment":{"href":"https://api.github.com/repos/apel/apel/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/apel/apel/pulls/50/commits"},"statuses":{"href":"https://api.github.com/repos/apel/apel/statuses/eb154f96fd539067033e99292018478744385726"}},"merged":true,"mergeable":null,"mergeable_state":"unknown","merged_by":{"login":"tofu-rocketry","id":4836233,"avatar_url":"https://avatars.githubusercontent.com/u/4836233?v=3","gravatar_id":"","url":"https://api.github.com/users/tofu-rocketry","html_url":"https://github.com/tofu-rocketry","followers_url":"https://api.github.com/users/tofu-rocketry/followers","following_url":"https://api.github.com/users/tofu-rocketry/following{/other_user}","gists_url":"https://api.github.com/users/tofu-rocketry/gists{/gist_id}","starred_url":"https://api.github.com/users/tofu-rocketry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tofu-rocketry/subscriptions","organizations_url":"https://api.github.com/users/tofu-rocketry/orgs","repos_url":"https://api.github.com/users/tofu-rocketry/repos","events_url":"https://api.github.com/users/tofu-rocketry/events{/privacy}","received_events_url":"https://api.github.com/users/tofu-rocketry/received_events","type":"User","site_admin":false},"comments":2,"review_comments":0,"commits":6,"additions":184,"deletions":180,"changed_files":25}}