{"action":"created","issue":{"url":"https://api.github.com/repos/uqfoundation/dill/issues/198","repository_url":"https://api.github.com/repos/uqfoundation/dill","labels_url":"https://api.github.com/repos/uqfoundation/dill/issues/198/labels{/name}","comments_url":"https://api.github.com/repos/uqfoundation/dill/issues/198/comments","events_url":"https://api.github.com/repos/uqfoundation/dill/issues/198/events","html_url":"https://github.com/uqfoundation/dill/pull/198","id":196233551,"number":198,"title":"Pytest and CI","user":{"login":"Peque","id":526577,"avatar_url":"https://avatars.githubusercontent.com/u/526577?v=3","gravatar_id":"","url":"https://api.github.com/users/Peque","html_url":"https://github.com/Peque","followers_url":"https://api.github.com/users/Peque/followers","following_url":"https://api.github.com/users/Peque/following{/other_user}","gists_url":"https://api.github.com/users/Peque/gists{/gist_id}","starred_url":"https://api.github.com/users/Peque/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Peque/subscriptions","organizations_url":"https://api.github.com/users/Peque/orgs","repos_url":"https://api.github.com/users/Peque/repos","events_url":"https://api.github.com/users/Peque/events{/privacy}","received_events_url":"https://api.github.com/users/Peque/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-12-17T17:43:34Z","updated_at":"2016-12-17T17:48:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/uqfoundation/dill/pulls/198","html_url":"https://github.com/uqfoundation/dill/pull/198","diff_url":"https://github.com/uqfoundation/dill/pull/198.diff","patch_url":"https://github.com/uqfoundation/dill/pull/198.patch"},"body":"This pull request implements initial `pytest` compatibility and CI with Travis.\r\n\r\nTests are executed with `tox`, meaning that the user can very easily reproduce all test cases (with different Python interpreters) locally and easily. `tox` calls `pytest` to run those test files that have been adapted. Then, the rest of the files, are tested as before (simply executing the test file with the interpreter).\r\n\r\nIt is a step forward to CI and `pytest` integration. Although it is not complete, I think it could be merged as-is (I don't see any downsides).\r\n\r\nLocally, I have already adapted most of the test files to be `pytest` compatible. However, they all fail at least at one point. Therefore, I would need more time to fix them and, probably, discuss them with you @mmckerns to understand what the test is doing and why is now failing.\r\n\r\nAny comments suggestions?"},"comment":{"url":"https://api.github.com/repos/uqfoundation/dill/issues/comments/267776349","html_url":"https://github.com/uqfoundation/dill/pull/198#issuecomment-267776349","issue_url":"https://api.github.com/repos/uqfoundation/dill/issues/198","id":267776349,"user":{"login":"Peque","id":526577,"avatar_url":"https://avatars.githubusercontent.com/u/526577?v=3","gravatar_id":"","url":"https://api.github.com/users/Peque","html_url":"https://github.com/Peque","followers_url":"https://api.github.com/users/Peque/followers","following_url":"https://api.github.com/users/Peque/following{/other_user}","gists_url":"https://api.github.com/users/Peque/gists{/gist_id}","starred_url":"https://api.github.com/users/Peque/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Peque/subscriptions","organizations_url":"https://api.github.com/users/Peque/orgs","repos_url":"https://api.github.com/users/Peque/repos","events_url":"https://api.github.com/users/Peque/events{/privacy}","received_events_url":"https://api.github.com/users/Peque/received_events","type":"User","site_admin":false},"created_at":"2016-12-17T17:48:45Z","updated_at":"2016-12-17T17:48:45Z","body":"Related issues/pull requests: #3, #139, #162, #191, #198\r\n\r\nAlso #114 (just for the inclusion of a `MANIFEST.in` file, which was necessary to run the tests)."}}
{"actor":{"display_login":"Peque"}}