Possible redundant import in application.py #207

Open
adityaathalye opened this Issue Jan 21, 2013 · 2 comments

2 participants

@adityaathalye

[Warning] i'm a Python beginner.

I'm reading webpy code and looking at this file: application.py

i understand the import on line 5, but why is 'webapi' imported again in line 6? Whereas the alias 'web' from line 5 gets used everywhere, 'webapi' is not used to invoke even a single method or attribute in the file

@djtrack16

@adityaathalye -- seems like a redundant import to me. Unless, we want to know explicitly when we are using the webapi module. I'm not sure why we'd want to do that though. I will submit a pull request with the change.

@djtrack16

this issue is fixed in latest repo, the fix just hasn't been approved into the master yet, for some reason. I just cloned the repo, and issue was resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment