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

Avoid adding `(imported from API)` to workflows #4434

Merged
merged 3 commits into from Aug 17, 2017

Conversation

Projects
None yet
3 participants
@chambm
Copy link
Contributor

commented Aug 16, 2017

It's kind of pointless and hinders setting up a Galaxy instance (e.g. from Docker) programmatically.

chambm added some commits Aug 16, 2017

Avoid adding `(imported from API)` to workflows
This should fail a few tests which I will fix with additional commits. Doing this from GitHub...
@bgruening

This comment has been minimized.

Copy link
Member

commented Aug 17, 2017

This is also slightly annoying in all training containers. Thanks for the PR.

@bgruening bgruening merged commit 008a1d8 into galaxyproject:dev Aug 17, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
lgtm analysis: JavaScript No alert changes
Details
@nsoranzo

This comment has been minimized.

Copy link
Member

commented Aug 17, 2017

It seems this broke API workflow tests, PR to fix them is coming as soon tests finish on my laptop.

nsoranzo added a commit to nsoranzo/galaxy that referenced this pull request Aug 17, 2017

dannon added a commit that referenced this pull request Aug 17, 2017

@chambm chambm deleted the chambm:patch-1 branch Aug 22, 2017

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.