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

Improve the Falco wrapper #6116

Merged
merged 2 commits into from
Jun 30, 2024
Merged

Improve the Falco wrapper #6116

merged 2 commits into from
Jun 30, 2024

Conversation

wm75
Copy link
Contributor

@wm75 wm75 commented Jun 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)
  • Mention FastQC in the description to make the tool findable when searching for the original
  • Add information about advantages and current limitations compared to FastQC to help section
  • Do not produce the results summary file by default, but only on demand
  • Simplify symlinking

Sorry, something went wrong.

- Mention FastQC in the description to make the tool findable when
  searching for the original
- Add information about advantages and current limitations compared to
  FastQC to help section
- Do not produce the results summary file by default, but only on demand
- Simplify symlinking
@wm75
Copy link
Contributor Author

wm75 commented Jun 30, 2024

Warning about mapped BAMs as input can be removed once smithlabcode/falco#59 is addressed and a new version is available.

@bgruening bgruening merged commit e1af25e into galaxyproject:main Jun 30, 2024
11 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.

None yet

2 participants