-
Notifications
You must be signed in to change notification settings - Fork 112
Relative imports #83
Comments
Then you should add that directory to your sys.path. --Matt Robenolt@mattrobenoltOn Mon, Jun 16, 2014 at 3:07 AM -0700, "AndersenTobias" [email protected] wrote: I'd like to change to relative imports as I want to run gargoyle inside my apps/ dir. —Reply to this email directly or view it on GitHub. |
@mattrobenolt Then nexus/modules.py L38 fails. Do I add media_root to gargoyle? |
@AndersenTobias I'm not sure what you're even asking me. What's |
@mattrobenolt Sorry for being unclear; I appreciate your patience. Can you please expand on why you think relative imports is a bad idea? I have a patch for it ready. |
@AndersenTobias PEP 328 suggests not using relative imports unless you really have to. Read https://www.python.org/dev/peps/pep-0328/ for an analysis. |
@giftig thanks! |
I'd like to change to relative imports as I want to run gargoyle inside my
apps/
dir.Let me know what you think about that.
The text was updated successfully, but these errors were encountered: