{"action":"closed","issue":{"url":"https://api.github.com/repos/ChenJesse/diningAppV2/issues/26","labels_url":"https://api.github.com/repos/ChenJesse/diningAppV2/issues/26/labels{/name}","comments_url":"https://api.github.com/repos/ChenJesse/diningAppV2/issues/26/comments","events_url":"https://api.github.com/repos/ChenJesse/diningAppV2/issues/26/events","html_url":"https://github.com/ChenJesse/diningAppV2/issues/26","id":89216603,"number":26,"title":"Create a better relative path that fosters clear understanding","user":{"login":"randytung","id":11008644,"avatar_url":"https://avatars.githubusercontent.com/u/11008644?v=3","gravatar_id":"","url":"https://api.github.com/users/randytung","html_url":"https://github.com/randytung","followers_url":"https://api.github.com/users/randytung/followers","following_url":"https://api.github.com/users/randytung/following{/other_user}","gists_url":"https://api.github.com/users/randytung/gists{/gist_id}","starred_url":"https://api.github.com/users/randytung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/randytung/subscriptions","organizations_url":"https://api.github.com/users/randytung/orgs","repos_url":"https://api.github.com/users/randytung/repos","events_url":"https://api.github.com/users/randytung/events{/privacy}","received_events_url":"https://api.github.com/users/randytung/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"randytung","id":11008644,"avatar_url":"https://avatars.githubusercontent.com/u/11008644?v=3","gravatar_id":"","url":"https://api.github.com/users/randytung","html_url":"https://github.com/randytung","followers_url":"https://api.github.com/users/randytung/followers","following_url":"https://api.github.com/users/randytung/following{/other_user}","gists_url":"https://api.github.com/users/randytung/gists{/gist_id}","starred_url":"https://api.github.com/users/randytung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/randytung/subscriptions","organizations_url":"https://api.github.com/users/randytung/orgs","repos_url":"https://api.github.com/users/randytung/repos","events_url":"https://api.github.com/users/randytung/events{/privacy}","received_events_url":"https://api.github.com/users/randytung/received_events","type":"User","site_admin":false},"milestone":null,"comments":0,"created_at":"2015-06-18T07:02:46Z","updated_at":"2015-06-20T21:54:59Z","closed_at":"2015-06-20T21:54:59Z","body":"suchan - \"I recommend that you put the path generating code in a method. Methods are good not only for making things modular but for labeling.\r\n\r\nImagine that in 6 months time you are reading this code again and you see the above 4 lines of code to get the projectroot. Are you actually trying to get the project root dir or are you starting from the root dir and finding something else? I don't know what you are doing and you also will have little understanding of what you were trying to do.\r\n\r\nIf you you put the path generating code in a method, you could instead call sys.path.append(getSomeImportantDir()) which (1) makes it clear what directory this is supposed to be (2) The steps are much clearer now that you know your endgoal\""}}