Skip to content

update delly#7789

Open
bgruening wants to merge 4 commits intogalaxyproject:mainfrom
bgruening:delly
Open

update delly#7789
bgruening wants to merge 4 commits intogalaxyproject:mainfrom
bgruening:delly

Conversation

@bgruening
Copy link
Member

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)

There are two labels that allow to ignore specific (false positive) tool linter errors:

  • skip-version-check: Use it if only a subset of the tools has been updated in a suite.
  • skip-url-check: Use it if github CI sees 403 errors, but the URLs work.

<expand macro="exclude"/>
</section>
<section name="discovery" title="Discovery options" expanded="true">
<param name="mapqual" type="integer" value="1" label="Set minimum paired-end mapping quality" help="(--map-qual)"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

If you keep the name attribute (an name in the command section) as it is workflows do not need updates.

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't want to keep the name attribute see my other PR :)

|& tee '$out_log'
#end if
]]></token>
<token name="@THREADS@">--threads "\${GALAXY_SLOTS:-1}"</token>
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add double quotes

Copy link
Member Author

Choose a reason for hiding this comment

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

I did, or what is missing?

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.

2 participants