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

updated snpeff version, fixed gb parsing py script #6387

Merged
merged 19 commits into from
Oct 21, 2024

Conversation

nekrut
Copy link
Contributor

@nekrut nekrut commented Sep 30, 2024

FOR CONTRIBUTOR:

  • I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • License permits unrestricted use (educational + commercial)
  • This PR adds a new tool or tool collection
  • This PR updates an existing tool or tool collection
  • This PR does something else (explain below)

@nekrut nekrut added the wip label Sep 30, 2024
@wm75
Copy link
Contributor

wm75 commented Oct 1, 2024

@nekrut there is an older, abandoned attempt in #4424
Feel free to take over whatever you think is useful from it.

@nekrut nekrut enabled auto-merge October 18, 2024 15:02
@bgruening bgruening changed the title updated snpeff version, fixed gb parsing py script. WIP updated snpeff version, fixed gb parsing py script Oct 18, 2024
tool_collections/snpeff/snpEff.xml Outdated Show resolved Hide resolved
tool_collections/snpeff/snpEff_databases.xml Outdated Show resolved Hide resolved
tool_collections/snpeff/snpEff_download.xml Outdated Show resolved Hide resolved
tool_collections/snpeff/snpEff_create_db.xml Outdated Show resolved Hide resolved
tool_collections/snpeff/snpEff_databases.xml Outdated Show resolved Hide resolved
tool_collections/snpeff/snpEff_macros.xml Outdated Show resolved Hide resolved
tool_collections/snpeff/snpeff_get_chr_names.xml Outdated Show resolved Hide resolved
tool_collections/snpeff/snpEff_create_db.xml Outdated Show resolved Hide resolved
tool_collections/snpeff/snpEff.xml Show resolved Hide resolved
nekrut and others added 4 commits October 18, 2024 15:42
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
nekrut and others added 4 commits October 18, 2024 15:44
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
@@ -1,11 +1,11 @@
<tool id="snpEff_build_gb" name="SnpEff build:" version="@WRAPPER_VERSION@.galaxy6" profile="22.01">
<tool id="snpEff_build_gb" name="SnpEff build:" version="@SNPEFF_VERSION@+galaxy@WRAPPER_VERSION@" profile="23.0">
Copy link
Member

Choose a reason for hiding this comment

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

instead of SNPEFF_VERSION could you please use TOOL_VERSION - https://galaxy-iuc-standards.readthedocs.io/en/latest/best_practices/tool_xml.html#tool-versions and also use @VERSION_SUFFIX@

this will help our bot with autoupdates.

@nekrut nekrut merged commit 4cc10cf into galaxyproject:main Oct 21, 2024
14 checks passed
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.

4 participants