{"action":"created","issue":{"url":"https://api.github.com/repos/dask/dask-jobqueue/issues/106","repository_url":"https://api.github.com/repos/dask/dask-jobqueue","labels_url":"https://api.github.com/repos/dask/dask-jobqueue/issues/106/labels{/name}","comments_url":"https://api.github.com/repos/dask/dask-jobqueue/issues/106/comments","events_url":"https://api.github.com/repos/dask/dask-jobqueue/issues/106/events","html_url":"https://github.com/dask/dask-jobqueue/issues/106","id":345061791,"node_id":"MDU6SXNzdWUzNDUwNjE3OTE=","number":106,"title":"Add flake8 to tests or black to CI","user":{"login":"mrocklin","id":306380,"node_id":"MDQ6VXNlcjMwNjM4MA==","avatar_url":"https://avatars3.githubusercontent.com/u/306380?v=4","gravatar_id":"","url":"https://api.github.com/users/mrocklin","html_url":"https://github.com/mrocklin","followers_url":"https://api.github.com/users/mrocklin/followers","following_url":"https://api.github.com/users/mrocklin/following{/other_user}","gists_url":"https://api.github.com/users/mrocklin/gists{/gist_id}","starred_url":"https://api.github.com/users/mrocklin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrocklin/subscriptions","organizations_url":"https://api.github.com/users/mrocklin/orgs","repos_url":"https://api.github.com/users/mrocklin/repos","events_url":"https://api.github.com/users/mrocklin/events{/privacy}","received_events_url":"https://api.github.com/users/mrocklin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-07-27T01:46:27Z","updated_at":"2018-07-28T13:51:13Z","closed_at":null,"author_association":"MEMBER","body":"Currently it looks like PRs can pass tests without passing flake8 tests.  Do we want to start enforcing style?"},"comment":{"url":"https://api.github.com/repos/dask/dask-jobqueue/issues/comments/408609289","html_url":"https://github.com/dask/dask-jobqueue/issues/106#issuecomment-408609289","issue_url":"https://api.github.com/repos/dask/dask-jobqueue/issues/106","id":408609289,"node_id":"MDEyOklzc3VlQ29tbWVudDQwODYwOTI4OQ==","user":{"login":"guillaumeeb","id":17138587,"node_id":"MDQ6VXNlcjE3MTM4NTg3","avatar_url":"https://avatars0.githubusercontent.com/u/17138587?v=4","gravatar_id":"","url":"https://api.github.com/users/guillaumeeb","html_url":"https://github.com/guillaumeeb","followers_url":"https://api.github.com/users/guillaumeeb/followers","following_url":"https://api.github.com/users/guillaumeeb/following{/other_user}","gists_url":"https://api.github.com/users/guillaumeeb/gists{/gist_id}","starred_url":"https://api.github.com/users/guillaumeeb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/guillaumeeb/subscriptions","organizations_url":"https://api.github.com/users/guillaumeeb/orgs","repos_url":"https://api.github.com/users/guillaumeeb/repos","events_url":"https://api.github.com/users/guillaumeeb/events{/privacy}","received_events_url":"https://api.github.com/users/guillaumeeb/received_events","type":"User","site_admin":false},"created_at":"2018-07-28T13:51:13Z","updated_at":"2018-07-28T13:51:13Z","author_association":"COLLABORATOR","body":"I remember asking myself if the none.sh job_script function would show something in Travis if flake checks failed.\r\n\r\nThe thing that troubled me is that we are running flake8 and py.test in one Travis operation. Maybe we should add a && between the two calls to take into account both return codes?"}}
{"actor":{"display_login":"guillaumeeb"}}