-
Notifications
You must be signed in to change notification settings - Fork 126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
change imports from spreg #22
Comments
This only applies to spreg dependencies, or needs to be changed throughout? @ljwolf |
This needs to be done everywhere, but I think the only place you do it is in this I appreciate the change; we're trying to cut down on the size of some powerful |
I see, so it only applies to |
No; it seems to only choke on: import spreg.user_ouput as USER but doesn't choke on from spreg import user_output as USER or from spreg.user_output import check_X |
Ok, then I think #24 satisfies this. Will merge and close. |
We're having some trouble with a few imports here.
If it's possible, can you switch from
import spreg.user_output as USER
tofrom spreg import user_output as USER
? For some reason we're not sure of, our build chokes when we have imports of the general pattern:import package.module_name as module_alias
instead of
from package import module_name as module_alias
We need to change this in the submodule contract, but it'll be present in future releases.
The text was updated successfully, but these errors were encountered: