Skip to content
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

Don't load tools from paths that start with `.` or `_` #3201

Merged

Conversation

Projects
None yet
2 participants
@mvdbeek
Copy link
Member

commented Nov 21, 2016

If planemo descends into .venv or _conda folders, startup can easily take 10-20 minutes.

Ran into this while developing a tool for which I also had a local .conda and .venv environment.

@mvdbeek mvdbeek added this to the 17.01 milestone Nov 21, 2016

@bgruening bgruening merged commit 78b533d into galaxyproject:dev Nov 21, 2016

3 checks passed

api test Build finished. 230 tests run, 0 skipped, 0 failed.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
toolshed test Build finished. 580 tests run, 0 skipped, 0 failed.
Details
@bgruening

This comment has been minimized.

Copy link
Member

commented Nov 21, 2016

Thanks!

@mvdbeek mvdbeek deleted the mvdbeek:do_not_load_tools_from_hidden_paths branch Nov 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.