{"action":"created","issue":{"url":"https://api.github.com/repos/hypothesis/h/issues/3148","repository_url":"https://api.github.com/repos/hypothesis/h","labels_url":"https://api.github.com/repos/hypothesis/h/issues/3148/labels{/name}","comments_url":"https://api.github.com/repos/hypothesis/h/issues/3148/comments","events_url":"https://api.github.com/repos/hypothesis/h/issues/3148/events","html_url":"https://github.com/hypothesis/h/pull/3148","id":144260612,"number":3148,"title":"Remove pep257 config from setup.cfg","user":{"login":"seanh","id":22498,"avatar_url":"https://avatars.githubusercontent.com/u/22498?v=3","gravatar_id":"","url":"https://api.github.com/users/seanh","html_url":"https://github.com/seanh","followers_url":"https://api.github.com/users/seanh/followers","following_url":"https://api.github.com/users/seanh/following{/other_user}","gists_url":"https://api.github.com/users/seanh/gists{/gist_id}","starred_url":"https://api.github.com/users/seanh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seanh/subscriptions","organizations_url":"https://api.github.com/users/seanh/orgs","repos_url":"https://api.github.com/users/seanh/repos","events_url":"https://api.github.com/users/seanh/events{/privacy}","received_events_url":"https://api.github.com/users/seanh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-29T13:13:24Z","updated_at":"2016-03-29T13:13:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hypothesis/h/pulls/3148","html_url":"https://github.com/hypothesis/h/pull/3148","diff_url":"https://github.com/hypothesis/h/pull/3148.diff","patch_url":"https://github.com/hypothesis/h/pull/3148.patch"},"body":"This tool has actually been renamed to pydocstyle now, but in any case\r\nwe don't run pep257/pydocstyle directly in h, we run it indirectly via\r\nthe prospector tool, and when run by prospector pep257 is configured\r\nseparately in the .prospector.yaml (and the pep257 config in\r\n.prospector.yaml was in conflict with the one in setup.cfg).\r\n\r\nI didn't copy the ignore = D202 from setup.cfg into .prospector.yaml as\r\nI don't see why we should ignore this rule.\r\n\r\nI also didn't copy the explain = true because it doesn't do anything,\r\nthis is a pep257/pydocstyle command line option that isn't available in\r\nthe config file."},"comment":{"url":"https://api.github.com/repos/hypothesis/h/issues/comments/202886341","html_url":"https://github.com/hypothesis/h/pull/3148#issuecomment-202886341","issue_url":"https://api.github.com/repos/hypothesis/h/issues/3148","id":202886341,"user":{"login":"codecov-io","id":8655789,"avatar_url":"https://avatars.githubusercontent.com/u/8655789?v=3","gravatar_id":"","url":"https://api.github.com/users/codecov-io","html_url":"https://github.com/codecov-io","followers_url":"https://api.github.com/users/codecov-io/followers","following_url":"https://api.github.com/users/codecov-io/following{/other_user}","gists_url":"https://api.github.com/users/codecov-io/gists{/gist_id}","starred_url":"https://api.github.com/users/codecov-io/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codecov-io/subscriptions","organizations_url":"https://api.github.com/users/codecov-io/orgs","repos_url":"https://api.github.com/users/codecov-io/repos","events_url":"https://api.github.com/users/codecov-io/events{/privacy}","received_events_url":"https://api.github.com/users/codecov-io/received_events","type":"User","site_admin":false},"created_at":"2016-03-29T13:13:25Z","updated_at":"2016-03-29T13:13:25Z","body":"## [Current coverage][1] is `69.94%`\n> Merging **#3148** into **master** will not affect coverage as of [`f47dec0`][3]\n\n\n```diff\n@@            master   #3148   diff @@\n======================================\n  Files          110     110       \n  Stmts         4369    4369       \n  Branches       512     512       \n  Methods          0       0       \n======================================\n  Hit           3056    3056       \n  Partial         83      83       \n  Missed        1230    1230       \n```\n\n> Review entire [Coverage Diff][4] as of [`f47dec0`][3]\n\n\n[1]: https://codecov.io/github/hypothesis/h?ref=f47dec0f6e5150cdc1b216b86c86a8c332aebbf3\n[2]: https://codecov.io/github/hypothesis/h/features/suggestions?ref=f47dec0f6e5150cdc1b216b86c86a8c332aebbf3\n[3]: https://codecov.io/github/hypothesis/h/commit/f47dec0f6e5150cdc1b216b86c86a8c332aebbf3\n[4]: https://codecov.io/github/hypothesis/h/compare/b35cd14b0fdda2a3eb2923604db167fccac388a9...f47dec0f6e5150cdc1b216b86c86a8c332aebbf3\n\n> Powered by [Codecov](https://codecov.io). Updated on successful CI builds."}}