{"action":"created","issue":{"url":"https://api.github.com/repos/Cloudef/buildhck/issues/5","labels_url":"https://api.github.com/repos/Cloudef/buildhck/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/Cloudef/buildhck/issues/5/comments","events_url":"https://api.github.com/repos/Cloudef/buildhck/issues/5/events","html_url":"https://github.com/Cloudef/buildhck/pull/5","id":73768378,"number":5,"title":"refactor: WIP","user":{"login":"buhman","id":728108,"avatar_url":"https://avatars.githubusercontent.com/u/728108?v=3","gravatar_id":"","url":"https://api.github.com/users/buhman","html_url":"https://github.com/buhman","followers_url":"https://api.github.com/users/buhman/followers","following_url":"https://api.github.com/users/buhman/following{/other_user}","gists_url":"https://api.github.com/users/buhman/gists{/gist_id}","starred_url":"https://api.github.com/users/buhman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/buhman/subscriptions","organizations_url":"https://api.github.com/users/buhman/orgs","repos_url":"https://api.github.com/users/buhman/repos","events_url":"https://api.github.com/users/buhman/events{/privacy}","received_events_url":"https://api.github.com/users/buhman/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-06T23:25:23Z","updated_at":"2015-05-06T23:34:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Cloudef/buildhck/pulls/5","html_url":"https://github.com/Cloudef/buildhck/pull/5","diff_url":"https://github.com/Cloudef/buildhck/pull/5.diff","patch_url":"https://github.com/Cloudef/buildhck/pull/5.patch"},"body":"Partially fixes #4 and possibly #3, depending on what exactly you mean there.\r\n\r\nNot done:\r\n\r\n* packaging for `buildhck.client`\r\n* pkgbuild (perhaps like what I did with [pb-git](https://aur.archlinux.org/packages/pb-git), only less painful)\r\n* configuration\r\n* finish making the testsuite less horrid (we don't actually care about testing the wsgi stack)\r\n\r\nRFC:\r\n\r\n* can I replace all graphics (other than favicon, because browsers suck) with vector equivalents?\r\n* do you agree/accept the unitest -> pytest thing?\r\n* I'd like to replace `authorization.def` with a yaml config loaded from xdg basedir\r\n* I noticed before I touched things, you had zero external dependencies, are you ok instead with needing:\r\n\r\n```\r\npip install -r requirements.txt\r\n```\r\n\r\n…to grab all the things?"},"comment":{"url":"https://api.github.com/repos/Cloudef/buildhck/issues/comments/99643875","html_url":"https://github.com/Cloudef/buildhck/pull/5#issuecomment-99643875","issue_url":"https://api.github.com/repos/Cloudef/buildhck/issues/5","id":99643875,"user":{"login":"Cloudef","id":480330,"avatar_url":"https://avatars.githubusercontent.com/u/480330?v=3","gravatar_id":"","url":"https://api.github.com/users/Cloudef","html_url":"https://github.com/Cloudef","followers_url":"https://api.github.com/users/Cloudef/followers","following_url":"https://api.github.com/users/Cloudef/following{/other_user}","gists_url":"https://api.github.com/users/Cloudef/gists{/gist_id}","starred_url":"https://api.github.com/users/Cloudef/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cloudef/subscriptions","organizations_url":"https://api.github.com/users/Cloudef/orgs","repos_url":"https://api.github.com/users/Cloudef/repos","events_url":"https://api.github.com/users/Cloudef/events{/privacy}","received_events_url":"https://api.github.com/users/Cloudef/received_events","type":"User","site_admin":false},"created_at":"2015-05-06T23:34:34Z","updated_at":"2015-05-06T23:34:34Z","body":"#3 means uploading and showing artifacts on the server side.\r\n\r\n> can I replace all graphics (other than favicon, because browsers suck) with vector equivalents?\r\n\r\nYes\r\n\r\n>do you agree with the unitest -> pytest thing?\r\n\r\nWhat are the pros for this?\r\n\r\n>I'd like to replace authorization.def with a yaml config loaded from xdg basedir\r\n\r\nSure.\r\n\r\n>I noticed before I touched things, you had zero external dependencies, are you ok instead with needing\r\n\r\nLess deps the better, but I'm fine with this as long as the setup is easy. Pip did allow user python path right as well? This should be fine.\r\n\r\n\r\nI'll look through the commits in this PR tomorrow though."}}