{"action":"created","issue":{"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok-client/issues/60","labels_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok-client/issues/60/labels{/name}","comments_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok-client/issues/60/comments","events_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok-client/issues/60/events","html_url":"https://github.com/Cal-CS-61A-Staff/ok-client/pull/60","id":59687211,"number":60,"title":"Fixed import so auth can be used through git submodules.","user":{"login":"moowiz","id":1504248,"avatar_url":"https://avatars.githubusercontent.com/u/1504248?v=3","gravatar_id":"","url":"https://api.github.com/users/moowiz","html_url":"https://github.com/moowiz","followers_url":"https://api.github.com/users/moowiz/followers","following_url":"https://api.github.com/users/moowiz/following{/other_user}","gists_url":"https://api.github.com/users/moowiz/gists{/gist_id}","starred_url":"https://api.github.com/users/moowiz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moowiz/subscriptions","organizations_url":"https://api.github.com/users/moowiz/orgs","repos_url":"https://api.github.com/users/moowiz/repos","events_url":"https://api.github.com/users/moowiz/events{/privacy}","received_events_url":"https://api.github.com/users/moowiz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-03T18:52:39Z","updated_at":"2015-03-03T21:16:32Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok-client/pulls/60","html_url":"https://github.com/Cal-CS-61A-Staff/ok-client/pull/60","diff_url":"https://github.com/Cal-CS-61A-Staff/ok-client/pull/60.diff","patch_url":"https://github.com/Cal-CS-61A-Staff/ok-client/pull/60.patch"},"body":"I'm trying to use ok-client as a git submodule in the ok-scripts repo for authentication. When I would try to import auth, I get the following. \r\n\r\n```\r\nTraceback (most recent call last):\r\n  File \"<stdin>\", line 1, in <module>\r\n  File \"/Users/moowiz/src/ok-scripts/ok_client/client/utils/auth.py\", line 3, in <module>\r\n    from client.utils.sanction import Client\r\nImportError: No module named 'client.utils.sanction'\r\n```\r\n\r\nAdding this relative import fixes this issue."},"comment":{"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok-client/issues/comments/77036216","html_url":"https://github.com/Cal-CS-61A-Staff/ok-client/pull/60#issuecomment-77036216","issue_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok-client/issues/60","id":77036216,"user":{"login":"Some3141","id":1359998,"avatar_url":"https://avatars.githubusercontent.com/u/1359998?v=3","gravatar_id":"","url":"https://api.github.com/users/Some3141","html_url":"https://github.com/Some3141","followers_url":"https://api.github.com/users/Some3141/followers","following_url":"https://api.github.com/users/Some3141/following{/other_user}","gists_url":"https://api.github.com/users/Some3141/gists{/gist_id}","starred_url":"https://api.github.com/users/Some3141/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Some3141/subscriptions","organizations_url":"https://api.github.com/users/Some3141/orgs","repos_url":"https://api.github.com/users/Some3141/repos","events_url":"https://api.github.com/users/Some3141/events{/privacy}","received_events_url":"https://api.github.com/users/Some3141/received_events","type":"User","site_admin":false},"created_at":"2015-03-03T21:16:32Z","updated_at":"2015-03-03T21:16:32Z","body":"I think this one should be okay since sanction is only used for auth, so there's a strong binding here. Is there a particular reason for not liking relative imports? The absolute imports also assume you're in a certain directory as well, which is worse for usability.\r\n\r\nThe main problem is that when you go inside the utils folder and try to import auth, then it complains. (or I think what @moowiz was trying to do was have the repo inside and import auth by doing something like 'import client.utils.auth' or the like.)"}}