-
Notifications
You must be signed in to change notification settings - Fork 982
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
Ignore .tox directory. Used to be ignored by default w/ flake8 appar… #2650
Conversation
…ntly, but not anymore.
…s with it still around. (nsoranzo's suggestion)
@@ -442,7 +442,7 @@ def setup_shed_tools_for_test(app, tmpdir, testing_migrated_tools, testing_insta | |||
tool_configs.remove(relative_migrated_tool_panel_config) | |||
for installed_tool_panel_config in INSTALLED_TOOL_PANEL_CONFIGS: | |||
tool_configs.append(installed_tool_panel_config) | |||
from galaxy import tools # noqa, delay import because this brings in so many modules for small tests | |||
from galaxy import tools # delay import because this brings in so many modules for small tests # noqa |
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.
Add : E402
at the end?
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.
Sure, good to specify. Going to look up the rest of them too.
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.
Cool, was doing the same! My ack found:
lib/galaxy/model/mapping.py
lib/galaxy/webapps/galaxy/controllers/dataset.py
lib/galaxy/webapps/galaxy/controllers/forms.py
lib/galaxy/webapps/galaxy/controllers/library_common.py
scripts/db_shell.py
… more #noqas to skip the precise relevant exception.
'_future_expose_api_raw_anonymous', | ||
'_future_expose_api_anonymous_and_sessionless', | ||
'_future_expose_api_raw_anonymous_and_sessionless', 'form', | ||
'FormBuilder', 'httpexceptions'] |
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.
Awesome, thanks a lot!
Do not pin when forward-merging to dev branch. Also ignore .tox/ and eggs/ as in galaxyproject#2650 .
…ently, but not anymore.