{"action":"created","issue":{"url":"https://api.github.com/repos/SatelliteQE/nailgun/issues/156","labels_url":"https://api.github.com/repos/SatelliteQE/nailgun/issues/156/labels{/name}","comments_url":"https://api.github.com/repos/SatelliteQE/nailgun/issues/156/comments","events_url":"https://api.github.com/repos/SatelliteQE/nailgun/issues/156/events","html_url":"https://github.com/SatelliteQE/nailgun/pull/156","id":96551374,"number":156,"title":"[DO NOT MERGE] Make `read`'s `ignore` argument easier to manipulate","user":{"login":"abalakh","id":11719030,"avatar_url":"https://avatars.githubusercontent.com/u/11719030?v=3","gravatar_id":"","url":"https://api.github.com/users/abalakh","html_url":"https://github.com/abalakh","followers_url":"https://api.github.com/users/abalakh/followers","following_url":"https://api.github.com/users/abalakh/following{/other_user}","gists_url":"https://api.github.com/users/abalakh/gists{/gist_id}","starred_url":"https://api.github.com/users/abalakh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abalakh/subscriptions","organizations_url":"https://api.github.com/users/abalakh/orgs","repos_url":"https://api.github.com/users/abalakh/repos","events_url":"https://api.github.com/users/abalakh/events{/privacy}","received_events_url":"https://api.github.com/users/abalakh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-07-22T11:56:37Z","updated_at":"2015-07-22T15:08:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SatelliteQE/nailgun/pulls/156","html_url":"https://github.com/SatelliteQE/nailgun/pull/156","diff_url":"https://github.com/SatelliteQE/nailgun/pull/156.diff","patch_url":"https://github.com/SatelliteQE/nailgun/pull/156.patch"},"body":"As per #134 discussion, together with @oshtaier made `read`'s `ignore` argument to be instance of set instead of tuple. Default value for `ignore` changed to `None`.\r\nThis will retain the two advantages:\r\n- None is immutable\r\n- Field should default to an empty iterable\r\nAnd introduce a new one:\r\n- Sets are much easier to manipulate, e.g. no need to use type castings to add/remove some elements\r\n\r\nCloses #134\r\n\r\n`[DO NOT MERGE]` prefix till i get test run results and validate everything's OK (~ 1 hour)"},"comment":{"url":"https://api.github.com/repos/SatelliteQE/nailgun/issues/comments/123753174","html_url":"https://github.com/SatelliteQE/nailgun/pull/156#issuecomment-123753174","issue_url":"https://api.github.com/repos/SatelliteQE/nailgun/issues/156","id":123753174,"user":{"login":"Ichimonji10","id":1012989,"avatar_url":"https://avatars.githubusercontent.com/u/1012989?v=3","gravatar_id":"","url":"https://api.github.com/users/Ichimonji10","html_url":"https://github.com/Ichimonji10","followers_url":"https://api.github.com/users/Ichimonji10/followers","following_url":"https://api.github.com/users/Ichimonji10/following{/other_user}","gists_url":"https://api.github.com/users/Ichimonji10/gists{/gist_id}","starred_url":"https://api.github.com/users/Ichimonji10/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ichimonji10/subscriptions","organizations_url":"https://api.github.com/users/Ichimonji10/orgs","repos_url":"https://api.github.com/users/Ichimonji10/repos","events_url":"https://api.github.com/users/Ichimonji10/events{/privacy}","received_events_url":"https://api.github.com/users/Ichimonji10/received_events","type":"User","site_admin":false},"created_at":"2015-07-22T15:08:20Z","updated_at":"2015-07-22T15:08:20Z","body":"@JacobCallahan I was unclear. :disappointed:"}}