GitHub PullRequestReviewCommentEvent at 2014-05-08 15:03:18
Repository: Ohouse
Actor: motine
Payload: {"comment":{"url":"https://api.github.com/repos/motine/Ohouse/pulls/comments/12433237","id":12433237,"diff_hunk":"@@ -36,3 +40,11 @@ def expand_amsoil_path(path):\n         return path\n     else:\n         return os.path.normpath(os.path.join(ROOT_PATH, path))\n+\n+\n+MONGO_CONFIG_PATH = expand_amsoil_path('deploy/config.json')\n+MONGO_CONFIG = json.load(open(MONGO_CONFIG_PATH))","path":"src/amsoil/config.py","position":25,"original_position":25,"commit_id":"46325e89618403733d485a5395db2e28ab07af46","original_commit_id":"46325e89618403733d485a5395db2e28ab07af46","user":{"login":"motine","id":1439041,"avatar_url":"https://avatars.githubusercontent.com/u/1439041?","gravatar_id":"1dcc728f0ed1028b0506547500726570","url":"https://api.github.com/users/motine","html_url":"https://github.com/motine","followers_url":"https://api.github.com/users/motine/followers","following_url":"https://api.github.com/users/motine/following{/other_user}","gists_url":"https://api.github.com/users/motine/gists{/gist_id}","starred_url":"https://api.github.com/users/motine/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/motine/subscriptions","organizations_url":"https://api.github.com/users/motine/orgs","repos_url":"https://api.github.com/users/motine/repos","events_url":"https://api.github.com/users/motine/events{/privacy}","received_events_url":"https://api.github.com/users/motine/received_events","type":"User","site_admin":false},"body":"did you consider the case if there is no such config file or not the right contents?\r\nCan you please make sure that if this is the case, that the system crashes with a decent error message.","created_at":"2014-05-08T15:03:18Z","updated_at":"2014-05-08T15:03:18Z","html_url":"https://github.com/motine/Ohouse/pull/51#discussion_r12433237","pull_request_url":"https://api.github.com/repos/motine/Ohouse/pulls/51","_links":{"self":{"href":"https://api.github.com/repos/motine/Ohouse/pulls/comments/12433237"},"html":{"href":"https://github.com/motine/Ohouse/pull/51#discussion_r12433237"},"pull_request":{"href":"https://api.github.com/repos/motine/Ohouse/pulls/51"}}}}