-
Notifications
You must be signed in to change notification settings - Fork 23
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
ImportError: cannot import name SortImports #2
Comments
Hi, thanks for reporting. @jpotterm is working on it |
Any progress on this error? I am getting the same thing in Sublime Text 3. |
Hi @dougwt no, not at all actually. Sorry for not reporting that back. You're welcome to provide the fix yourself? :) |
@thijsdezoete I've submitted a pull request with my proposed fixes: #11 Cheers! |
I'm not 100% sure how package control works... does a new version tag need to be created? It seems to still be installing the old version from Package Control. |
@dougwt Hm, last time I merged something, when I reinstalled, it was just the new version. I never really took the time to properly solve the versioning/tagging thing, I can have a look this Wednesday if you like. |
@thijsdezoete I think all that might be required is to tag the merged version as release 0.0.4 |
@dougwt Ah, ok, I pushed a new tag 0.0.4 now |
@thijsdezoete Sorry, it looks like I may have been mistaken about the tag thing. However, now that I know Package Control was installing the latest version, I've gone back to the docs and it looks like the issue might be caused due to the way Package Control packages its plugins. I've submitted a pull request (#12) that includes a file telling Package Control not to package the plugin. Hopefully that solves the issue! |
@dougwt did it? |
@thijsdezoete I was still getting the ImportError after all of that. However, since the plugin was installed to a directory I was able to edit it directly until finding an import statement that works. (#13) I'm not sure if the .no-sublime-package is required at this point, but hopefully this should return the plugin to working order. |
@dougwt Ok, so could we add that one to the repo, and then get this over with? |
You're quite welcome. Please accept Pull Request #13 and we'll see if that works! |
Everything seems to be working in Sublime Text 3 at last 👍 |
I get this error when installing it in sublime 3
The text was updated successfully, but these errors were encountered: