{"action":"created","issue":{"url":"https://api.github.com/repos/PSU-OIT-ARC/oregoninvasiveshotline/issues/19","labels_url":"https://api.github.com/repos/PSU-OIT-ARC/oregoninvasiveshotline/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/PSU-OIT-ARC/oregoninvasiveshotline/issues/19/comments","events_url":"https://api.github.com/repos/PSU-OIT-ARC/oregoninvasiveshotline/issues/19/events","html_url":"https://github.com/PSU-OIT-ARC/oregoninvasiveshotline/pull/19","id":121817950,"number":19,"title":"Use PIL instead of ImageMagick for *all* image operations","user":{"login":"kfarr2","id":7905314,"avatar_url":"https://avatars.githubusercontent.com/u/7905314?v=3","gravatar_id":"","url":"https://api.github.com/users/kfarr2","html_url":"https://github.com/kfarr2","followers_url":"https://api.github.com/users/kfarr2/followers","following_url":"https://api.github.com/users/kfarr2/following{/other_user}","gists_url":"https://api.github.com/users/kfarr2/gists{/gist_id}","starred_url":"https://api.github.com/users/kfarr2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kfarr2/subscriptions","organizations_url":"https://api.github.com/users/kfarr2/orgs","repos_url":"https://api.github.com/users/kfarr2/repos","events_url":"https://api.github.com/users/kfarr2/events{/privacy}","received_events_url":"https://api.github.com/users/kfarr2/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-12-12T00:17:56Z","updated_at":"2015-12-19T03:36:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/PSU-OIT-ARC/oregoninvasiveshotline/pulls/19","html_url":"https://github.com/PSU-OIT-ARC/oregoninvasiveshotline/pull/19","diff_url":"https://github.com/PSU-OIT-ARC/oregoninvasiveshotline/pull/19.diff","patch_url":"https://github.com/PSU-OIT-ARC/oregoninvasiveshotline/pull/19.patch"},"body":"I would use `resize()` but `thumbnail()` calculates the aspect ratio and calls `resize()` for us anyway."},"comment":{"url":"https://api.github.com/repos/PSU-OIT-ARC/oregoninvasiveshotline/issues/comments/165941066","html_url":"https://github.com/PSU-OIT-ARC/oregoninvasiveshotline/pull/19#issuecomment-165941066","issue_url":"https://api.github.com/repos/PSU-OIT-ARC/oregoninvasiveshotline/issues/19","id":165941066,"user":{"login":"wylee","id":751427,"avatar_url":"https://avatars.githubusercontent.com/u/751427?v=3","gravatar_id":"","url":"https://api.github.com/users/wylee","html_url":"https://github.com/wylee","followers_url":"https://api.github.com/users/wylee/followers","following_url":"https://api.github.com/users/wylee/following{/other_user}","gists_url":"https://api.github.com/users/wylee/gists{/gist_id}","starred_url":"https://api.github.com/users/wylee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wylee/subscriptions","organizations_url":"https://api.github.com/users/wylee/orgs","repos_url":"https://api.github.com/users/wylee/repos","events_url":"https://api.github.com/users/wylee/events{/privacy}","received_events_url":"https://api.github.com/users/wylee/received_events","type":"User","site_admin":false},"created_at":"2015-12-19T03:36:49Z","updated_at":"2015-12-19T03:36:49Z","body":"- The commit messages for both commits aren't formatted according to standards\r\n- `utils.resize_image` should be renamed first and then re-implemented\r\n- The docstring for `generate_thumbnail` was copied verbatim from an inline comment on an earlier version of the commit; it still says \"blah dee blah dee blah\"\r\n- `import os, sys` should be two imports\r\n- Stdlib imports should be alphabetized\r\n- There should be a blank line after stdlib imports and other imports\r\n- A small, simple function like `generate_thumbnail` doesn't need any blank lines\r\n- In `generate_thumbnail`, you should try the open in a different block from where you try the save; that way, you can catch different types of errors and log them appropriately\r\n\r\n"}}