{"action":"created","issue":{"url":"https://api.github.com/repos/pulp/pulp_openstack/issues/25","repository_url":"https://api.github.com/repos/pulp/pulp_openstack","labels_url":"https://api.github.com/repos/pulp/pulp_openstack/issues/25/labels{/name}","comments_url":"https://api.github.com/repos/pulp/pulp_openstack/issues/25/comments","events_url":"https://api.github.com/repos/pulp/pulp_openstack/issues/25/events","html_url":"https://github.com/pulp/pulp_openstack/pull/25","id":136426329,"number":25,"title":"remove pep257 requirement from tests","user":{"login":"seandst","id":441252,"avatar_url":"https://avatars.githubusercontent.com/u/441252?v=3","gravatar_id":"","url":"https://api.github.com/users/seandst","html_url":"https://github.com/seandst","followers_url":"https://api.github.com/users/seandst/followers","following_url":"https://api.github.com/users/seandst/following{/other_user}","gists_url":"https://api.github.com/users/seandst/gists{/gist_id}","starred_url":"https://api.github.com/users/seandst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seandst/subscriptions","organizations_url":"https://api.github.com/users/seandst/orgs","repos_url":"https://api.github.com/users/seandst/repos","events_url":"https://api.github.com/users/seandst/events{/privacy}","received_events_url":"https://api.github.com/users/seandst/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/pulp/pulp_openstack/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/pulp/pulp_openstack/labels/LGTM","name":"LGTM","color":"5319e7"}],"state":"open","locked":false,"assignee":{"login":"seandst","id":441252,"avatar_url":"https://avatars.githubusercontent.com/u/441252?v=3","gravatar_id":"","url":"https://api.github.com/users/seandst","html_url":"https://github.com/seandst","followers_url":"https://api.github.com/users/seandst/followers","following_url":"https://api.github.com/users/seandst/following{/other_user}","gists_url":"https://api.github.com/users/seandst/gists{/gist_id}","starred_url":"https://api.github.com/users/seandst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seandst/subscriptions","organizations_url":"https://api.github.com/users/seandst/orgs","repos_url":"https://api.github.com/users/seandst/repos","events_url":"https://api.github.com/users/seandst/events{/privacy}","received_events_url":"https://api.github.com/users/seandst/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2016-02-25T15:48:59Z","updated_at":"2016-02-25T18:38:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/pulp/pulp_openstack/pulls/25","html_url":"https://github.com/pulp/pulp_openstack/pull/25","diff_url":"https://github.com/pulp/pulp_openstack/pull/25.diff","patch_url":"https://github.com/pulp/pulp_openstack/pull/25.patch"},"body":"No other plugin uses this, and it's currently breaking due to a change\r\nin pep257 that would require yet another exception to be added. As far\r\nas I can tell, we've added exceptions for every error that pep257 might\r\ntell us about, so this change is effectively a no-op.\r\n\r\nI do think docstrings are good, and that we should have them, so I'm\r\nreferring to a story that I just made about getting docstrings added to\r\npulp. Discussions of the pro/cons of requiring docstrings should\r\nprobably happen there.\r\n\r\nFurthermore, this commit can be used as a reference for seeing how we\r\nintegrated pep257 once upon a time.\r\n\r\nre #1718\r\nhttps://pulp.plan.io/issues/1718"},"comment":{"url":"https://api.github.com/repos/pulp/pulp_openstack/issues/comments/188922407","html_url":"https://github.com/pulp/pulp_openstack/pull/25#issuecomment-188922407","issue_url":"https://api.github.com/repos/pulp/pulp_openstack/issues/25","id":188922407,"user":{"login":"rbarlow","id":354506,"avatar_url":"https://avatars.githubusercontent.com/u/354506?v=3","gravatar_id":"","url":"https://api.github.com/users/rbarlow","html_url":"https://github.com/rbarlow","followers_url":"https://api.github.com/users/rbarlow/followers","following_url":"https://api.github.com/users/rbarlow/following{/other_user}","gists_url":"https://api.github.com/users/rbarlow/gists{/gist_id}","starred_url":"https://api.github.com/users/rbarlow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rbarlow/subscriptions","organizations_url":"https://api.github.com/users/rbarlow/orgs","repos_url":"https://api.github.com/users/rbarlow/repos","events_url":"https://api.github.com/users/rbarlow/events{/privacy}","received_events_url":"https://api.github.com/users/rbarlow/received_events","type":"User","site_admin":false},"created_at":"2016-02-25T18:38:43Z","updated_at":"2016-02-25T18:38:43Z","body":"And LGTM is purple in this repo…?"}}