{"action":"created","issue":{"url":"https://api.github.com/repos/letsencrypt/lets-encrypt-preview/issues/186","labels_url":"https://api.github.com/repos/letsencrypt/lets-encrypt-preview/issues/186/labels{/name}","comments_url":"https://api.github.com/repos/letsencrypt/lets-encrypt-preview/issues/186/comments","events_url":"https://api.github.com/repos/letsencrypt/lets-encrypt-preview/issues/186/events","html_url":"https://github.com/letsencrypt/lets-encrypt-preview/pull/186","id":55371382,"number":186,"title":"Pylint 10/10","user":{"login":"kuba","id":106176,"avatar_url":"https://avatars.githubusercontent.com/u/106176?v=3","gravatar_id":"","url":"https://api.github.com/users/kuba","html_url":"https://github.com/kuba","followers_url":"https://api.github.com/users/kuba/followers","following_url":"https://api.github.com/users/kuba/following{/other_user}","gists_url":"https://api.github.com/users/kuba/gists{/gist_id}","starred_url":"https://api.github.com/users/kuba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kuba/subscriptions","organizations_url":"https://api.github.com/users/kuba/orgs","repos_url":"https://api.github.com/users/kuba/repos","events_url":"https://api.github.com/users/kuba/events{/privacy}","received_events_url":"https://api.github.com/users/kuba/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-24T13:49:45Z","updated_at":"2015-01-24T19:27:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/letsencrypt/lets-encrypt-preview/pulls/186","html_url":"https://github.com/letsencrypt/lets-encrypt-preview/pull/186","diff_url":"https://github.com/letsencrypt/lets-encrypt-preview/pull/186.diff","patch_url":"https://github.com/letsencrypt/lets-encrypt-preview/pull/186.patch"},"body":"Fixes #178.\r\n\r\nChanges to .pylintrc:\r\n- longer function/method names (especially important for unit tests)\r\n- docstrings unnecessary for unittest classes\r\n- `unused_` prefix for unused arguments\r\n- allow locally disabling pylint messages\r\n- disable warnings on \"fixme\" comments\r\n\r\nFixed remaining pylint errors and decided to locally disable `missing-docstring` to make Travis happily green :) This way only new offending code will fail. At the same time, we should strive to replace `# pylint: disable missing-docstring` with meaningful docstrings. Since you know the code the most, @jdkasten, would you be able to fix this? Provided PR is accepted I will create necessary tracking bug report."},"comment":{"url":"https://api.github.com/repos/letsencrypt/lets-encrypt-preview/issues/comments/71333386","html_url":"https://github.com/letsencrypt/lets-encrypt-preview/pull/186#issuecomment-71333386","issue_url":"https://api.github.com/repos/letsencrypt/lets-encrypt-preview/issues/186","id":71333386,"user":{"login":"ThomasWaldmann","id":356103,"avatar_url":"https://avatars.githubusercontent.com/u/356103?v=3","gravatar_id":"","url":"https://api.github.com/users/ThomasWaldmann","html_url":"https://github.com/ThomasWaldmann","followers_url":"https://api.github.com/users/ThomasWaldmann/followers","following_url":"https://api.github.com/users/ThomasWaldmann/following{/other_user}","gists_url":"https://api.github.com/users/ThomasWaldmann/gists{/gist_id}","starred_url":"https://api.github.com/users/ThomasWaldmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ThomasWaldmann/subscriptions","organizations_url":"https://api.github.com/users/ThomasWaldmann/orgs","repos_url":"https://api.github.com/users/ThomasWaldmann/repos","events_url":"https://api.github.com/users/ThomasWaldmann/events{/privacy}","received_events_url":"https://api.github.com/users/ThomasWaldmann/received_events","type":"User","site_admin":false},"created_at":"2015-01-24T19:27:22Z","updated_at":"2015-01-24T19:27:22Z","body":"yay! :)"}}