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

[CI] Update CI to include standalone wfs and other wfs #614

Open
kapsakcj opened this issue Sep 11, 2024 · 0 comments
Open

[CI] Update CI to include standalone wfs and other wfs #614

kapsakcj opened this issue Sep 11, 2024 · 0 comments

Comments

@kapsakcj
Copy link
Contributor

🆒

📌 Explain the Request

Would be good to add CI functionality to test the many standalone workflows so that bugs are caught prior to releases. Workflows like

  • ncbi_scrub_PE and SE
  • ncbi_amrfinderplus
  • sra_fetch
  • basespace_fetch
  • various Freyja workflows (plot, dashboard, FASTQ, update)
  • pangolin_update
  • vadr_update (very lower priority, I don't know if anyone even uses this one)
  • and many more

We will definitely need to prioritize which workflows that would benefit from this.

Adding these to do a simple miniwdl check in the CI would be a good first step. From there we can get more advanced and thorough and actually run the workflows with miniwdl and cromwell, test for existence of outputfiles, check hashes of those output files, etc., etc.

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

No branches or pull requests

1 participant