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

Remove unused export base mako #4661

Merged
merged 1 commit into from Sep 20, 2017

Conversation

Projects
None yet
3 participants
@guerler
Contributor

guerler commented Sep 20, 2017

Is this mako used anywhere? @martenson.

@dannon

This comment has been minimized.

Member

dannon commented Sep 20, 2017

@guerler Not seeing any current use of it. Looks like it should have been removed in ccbc3e0.

@guerler guerler added status/review and removed status/WIP labels Sep 20, 2017

@martenson

This comment has been minimized.

Member

martenson commented Sep 20, 2017

I see dead code. Good catch @guerler

@martenson martenson merged commit a25a76a into galaxyproject:dev Sep 20, 2017

2 of 6 checks passed

api test Test started.
Details
framework test Test started.
Details
lgtm analysis: JavaScript Running analyses for revisions
Details
toolshed test Test started.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
integration test Build finished. 46 tests run, 0 skipped, 0 failed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment