-
Notifications
You must be signed in to change notification settings - Fork 53
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
Convert to Absolute Imports [AARD-1737]
#1013
Conversation
[AARD-1737]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@HunterBarclay Looks like the formatter for the exporter broke. My best guess is that the version of isort
that runs on linux is behaving differently than the one for mac and windows. Might just need to disable it for now.
All files in this PR are formatting correctly.
Edit:
Check #1081 for the fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
general_imports.py
and all code that was running on some form ofimport *
.cmdDefs
inMarkingMenu.py
was not being reset, moving into an__init__
structure made this obvious.__init__
file to contain the global and saved state. This state will persist through restarts of the exporter.JIRA Issue