Skip to content

Select refgenie config based on Galaxy version#1187

Merged
mvdbeek merged 2 commits intogalaxyproject:masterfrom
mvdbeek:select_config_version_based_on_version_major
Sep 21, 2021
Merged

Select refgenie config based on Galaxy version#1187
mvdbeek merged 2 commits intogalaxyproject:masterfrom
mvdbeek:select_config_version_based_on_version_major

Conversation

@mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Sep 21, 2021

if available. Else default to current version (0.4).

Fixes https://github.com/galaxyproject/tools-iuc/runs/3654913135?check_suite_focus=true

@mvdbeek mvdbeek force-pushed the select_config_version_based_on_version_major branch from 6dc6b6e to f40faf1 Compare September 21, 2021 08:30
Copy link
Collaborator

@bernt-matthias bernt-matthias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wondering if we should setup a test for this, e.g. something like the mock planemo run during the setup stage in the IUC CI vs the latest Galaxy release.

One might also parametrize the planemo to use (fork and branch) in the CI action .. but this might be overkill.

Apart from this it looks good to me ..

@mvdbeek mvdbeek force-pushed the select_config_version_based_on_version_major branch from 0c8d673 to 0768932 Compare September 21, 2021 12:50
@jmchilton
Copy link
Member

Thanks for working through all this complexity. I wish we were deriving more benefit from refgenie.

@mvdbeek mvdbeek merged commit befb2a2 into galaxyproject:master Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants