-
Notifications
You must be signed in to change notification settings - Fork 290
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
toml is not required. Refactor import toml #566
Comments
please more information.
|
@hhatto My environment doesn't matter. I'm running But since you are curious. |
I just copy the autopep8.py file and the pycodestyle.py file into my project and use it/call it from there. I dont use pip for installing this. I manually extract those 2 specific files into my project from the whl/zip archives when I upgrade them from time to time. |
@hhatto if you comment out p = subprocess.Popen([sys.executable, # 'python',
_autopep8ScriptPath, # 'autopep8.py',
# '--first',
'--select=%s' % self.checkedErrorsAndWarnings, # Ex: '--select=E221,E222,E223,
# '--ignore=%s' % self.checkedErrorsAndWarnings, # Ex: '--ignore=E',
# '--show-pep8',
# '--max-line-length=120',
fileToCheck],
stdout=subprocess.PIPE,
stderr=subprocess.PIPE)
out, err = p.communicate() |
fix in latest master. 42aaa2a This change will be released in version 1.5.5. Thanks |
Yep that lazy import in the function will work fine. Thanks. :) |
toml import is not required and is throwing ImportError. This needs refactored so it throws an error when trying to access features that actually use it.
As you can see by the code, that it can be run on the cmd as a singlefile script(like pep8.py and pycodestyle.py is) and code added afterwards which makes it undoable/redoable in an editor.
Collapsible Content - Click to expand
The text was updated successfully, but these errors were encountered: