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

replace @VERSION@ with @TOOL_VERSION@ in gff_prot wrapper #4266

Merged

Conversation

cat-bro
Copy link
Contributor

@cat-bro cat-bro commented Nov 30, 2021

As part of Galaxy Australia's updates we have install gff_prot with version "@Version@+galaxy0". This is because in the macros the token has changed to @TOOL_VERSION. This is the only instance of @Version@ I could find in the /transit/ directory

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)

Copy link
Contributor

@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.

@cat-bro do you have the link to the PR that changed this? I would like to know why CI did not fail.

@bgruening bgruening merged commit ba889fa into galaxyproject:master Nov 30, 2021
@cat-bro
Copy link
Contributor Author

cat-bro commented Nov 30, 2021

@bernt-matthias #4223

@nsoranzo
Copy link
Member

nsoranzo commented Dec 1, 2021

In fact the CI found it but it's a warning:

Applying linter xml_order... WARNING
.. WARNING: Best practice violation [macros] elements should come before [xrefs]
Linting repository /home/runner/work/tools-iuc/tools-iuc/tools/transit
+Linting tool /home/runner/work/tools-iuc/tools-iuc/tools/transit/convert_gff.xml
Applying linter xml_order... WARNING
.. WARNING: Best practice violation [macros] elements should come before [xrefs]
Applying linter general... WARNING
.. WARNING: Tool version [@VERSION@+galaxy0] is not compliant with PEP 440.
Linting repository /home/runner/work/tools-iuc/tools-iuc/tools/transit
+Linting tool /home/runner/work/tools-iuc/tools-iuc/tools/transit/transit_gumbel.xml
Applying linter xml_order... WARNING
.. WARNING: Best practice violation [macros] elements should come before [xrefs]
Linting repository /home/runner/work/tools-iuc/tools-iuc/tools/transit
+Linting tool /home/runner/work/tools-iuc/tools-iuc/tools/transit/transit_resampling.xml
Applying linter xml_order... WARNING
.. WARNING: Best practice violation [macros] elements should come before [xrefs]
Linting repository /home/runner/work/tools-iuc/tools-iuc/tools/transit
+Linting tool /home/runner/work/tools-iuc/tools-iuc/tools/transit/transit_hmm.xml
Applying linter xml_order... WARNING
.. WARNING: Best practice violation [macros] elements should come before [xrefs]

@bernt-matthias
Copy link
Contributor

Thanks for looking this up @nsoranzo

Maybe these will help in the future galaxyproject/planemo-ci-action#23 galaxyproject/galaxy-tool-repository-template#8

But I'm a bit worried that setting the fail level to warn might make to much noise...

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