{"action":"created","issue":{"url":"https://api.github.com/repos/fedora-infra/anitya/issues/322","repository_url":"https://api.github.com/repos/fedora-infra/anitya","labels_url":"https://api.github.com/repos/fedora-infra/anitya/issues/322/labels{/name}","comments_url":"https://api.github.com/repos/fedora-infra/anitya/issues/322/comments","events_url":"https://api.github.com/repos/fedora-infra/anitya/issues/322/events","html_url":"https://github.com/fedora-infra/anitya/pull/322","id":168591348,"number":322,"title":"WIP: Initial pass at Py3k compatibility","user":{"login":"ncoghlan","id":1026649,"avatar_url":"https://avatars.githubusercontent.com/u/1026649?v=3","gravatar_id":"","url":"https://api.github.com/users/ncoghlan","html_url":"https://github.com/ncoghlan","followers_url":"https://api.github.com/users/ncoghlan/followers","following_url":"https://api.github.com/users/ncoghlan/following{/other_user}","gists_url":"https://api.github.com/users/ncoghlan/gists{/gist_id}","starred_url":"https://api.github.com/users/ncoghlan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ncoghlan/subscriptions","organizations_url":"https://api.github.com/users/ncoghlan/orgs","repos_url":"https://api.github.com/users/ncoghlan/repos","events_url":"https://api.github.com/users/ncoghlan/events{/privacy}","received_events_url":"https://api.github.com/users/ncoghlan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-08-01T08:18:08Z","updated_at":"2016-08-01T08:36:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/fedora-infra/anitya/pulls/322","html_url":"https://github.com/fedora-infra/anitya/pull/322","diff_url":"https://github.com/fedora-infra/anitya/pull/322.diff","patch_url":"https://github.com/fedora-infra/anitya/pull/322.patch"},"body":"- Initial automated update with python-modernize\r\n- Additional updates to get tests passing on 3.5\r\n\r\nTODO:\r\n  - check with pylint --py3k\r\n  - add tox configuration to test 2.6, 2.7 & 3.5\r\n  - conditional dep for python[3]-openid"},"comment":{"url":"https://api.github.com/repos/fedora-infra/anitya/issues/comments/236521182","html_url":"https://github.com/fedora-infra/anitya/pull/322#issuecomment-236521182","issue_url":"https://api.github.com/repos/fedora-infra/anitya/issues/322","id":236521182,"user":{"login":"ncoghlan","id":1026649,"avatar_url":"https://avatars.githubusercontent.com/u/1026649?v=3","gravatar_id":"","url":"https://api.github.com/users/ncoghlan","html_url":"https://github.com/ncoghlan","followers_url":"https://api.github.com/users/ncoghlan/followers","following_url":"https://api.github.com/users/ncoghlan/following{/other_user}","gists_url":"https://api.github.com/users/ncoghlan/gists{/gist_id}","starred_url":"https://api.github.com/users/ncoghlan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ncoghlan/subscriptions","organizations_url":"https://api.github.com/users/ncoghlan/orgs","repos_url":"https://api.github.com/users/ncoghlan/repos","events_url":"https://api.github.com/users/ncoghlan/events{/privacy}","received_events_url":"https://api.github.com/users/ncoghlan/received_events","type":"User","site_admin":false},"created_at":"2016-08-01T08:36:29Z","updated_at":"2016-08-01T08:36:29Z","body":"I mostly did this as a way of gaining some initial familiarity with the code base, but I also think it's a change worth working towards, and this gives some idea of the magnitude of updates needed.\r\n\r\nOverall, they're relatively minor, but the main one that gives me pause is the use of regular expressions to scrape page content - the current PR keeps that as a text domain operation, which should be fine in most cases, but the decode step needs to be checked to see how robust it is likely to be against text encoding errors when retrieving pages for scraping.\r\n\r\nNear term, though, my objective is to enable local testing and debugging under Python 3 without breaking deployment under Python 2, so more in-depth testing of the Unicode compatibility handling can wait for now."}}
{"actor":{"display_login":"ncoghlan"}}