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

[17.09] Fix ICM datatype sniffer config #5121

Merged
merged 1 commit into from Dec 4, 2017

Conversation

Projects
None yet
3 participants
@mvdbeek
Member

mvdbeek commented Dec 3, 2017

Probably a bad merge in #5108

@nsoranzo

This comment has been minimized.

Member

nsoranzo commented Dec 3, 2017

That's actually broken in 17.09 too, can you change the target branch?

Fix ICM datatype sniffer config
Probably a bad merge in #5108

@mvdbeek mvdbeek changed the base branch from dev to release_17.09 Dec 4, 2017

@mvdbeek mvdbeek changed the title from Fix ICM datatype sniffer config to [17.09] Fix ICM datatype sniffer config Dec 4, 2017

@mvdbeek mvdbeek removed the minor label Dec 4, 2017

@mvdbeek

This comment has been minimized.

Member

mvdbeek commented Dec 4, 2017

Thanks for checking @nsoranzo, I have updated the target branch.

@bgruening

This comment has been minimized.

Member

bgruening commented Dec 4, 2017

👍

@bgruening bgruening merged commit 22aa02f into galaxyproject:release_17.09 Dec 4, 2017

6 checks passed

api test Build finished. 293 tests run, 4 skipped, 0 failed.
Details
framework test Build finished. 162 tests run, 0 skipped, 0 failed.
Details
integration test Build finished. 46 tests run, 0 skipped, 0 failed.
Details
lgtm analysis: JavaScript No alert changes
Details
selenium test Build finished. No test results found.
Details
toolshed test Build finished. 579 tests run, 0 skipped, 0 failed.
Details

@mvdbeek mvdbeek deleted the mvdbeek:fix_icm_datatype_config branch Jun 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment