{"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":[{"id":45530887,"url":"https://api.github.com/repos/uqfoundation/dill/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":11,"created_at":"2016-12-17T17:43:34Z","updated_at":"2016-12-28T02:30:28Z","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. [See the results after setting up the repository with Travis](https://travis-ci.org/Peque/dill).\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/269415040","html_url":"https://github.com/uqfoundation/dill/pull/198#issuecomment-269415040","issue_url":"https://api.github.com/repos/uqfoundation/dill/issues/198","id":269415040,"user":{"login":"mmckerns","id":321534,"avatar_url":"https://avatars.githubusercontent.com/u/321534?v=3","gravatar_id":"","url":"https://api.github.com/users/mmckerns","html_url":"https://github.com/mmckerns","followers_url":"https://api.github.com/users/mmckerns/followers","following_url":"https://api.github.com/users/mmckerns/following{/other_user}","gists_url":"https://api.github.com/users/mmckerns/gists{/gist_id}","starred_url":"https://api.github.com/users/mmckerns/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmckerns/subscriptions","organizations_url":"https://api.github.com/users/mmckerns/orgs","repos_url":"https://api.github.com/users/mmckerns/repos","events_url":"https://api.github.com/users/mmckerns/events{/privacy}","received_events_url":"https://api.github.com/users/mmckerns/received_events","type":"User","site_admin":false},"created_at":"2016-12-28T02:30:28Z","updated_at":"2016-12-28T02:30:28Z","body":"@Peque: yeah sorry about the dissociated comments, I was trying something new and it didn't work as I expected."}}
{"actor":{"display_login":"mmckerns"}}