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

Run CI against blab/veme-2022 and zika-tutorial #1350

Open
jameshadfield opened this issue Dec 5, 2023 · 0 comments
Open

Run CI against blab/veme-2022 and zika-tutorial #1350

jameshadfield opened this issue Dec 5, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@jameshadfield
Copy link
Member

Currently we have (the extremely helpful) pathogen-repo-ci CI action (although it confusingly doesn't use the reusable workflow of the same name). This helps ensure changes to Augur don't inadvertently break our ~core pathogen builds. We should do the same for the VEME tutorial repo and the zika-tutorial workflow, which are often external users' first interactions with Nextstrain/Augur. The latter is essentially our quickstart guide.

Adding zika-tutorial to our CI should be easy.

Adding the VEME tutorial will be harder because it does not use a Snakefile. Rather than add one, we could extract the salient commands programatically from the README. Similarly to how individual build repos run their own CI (often using the reusable pathogen-repo-ci workflow), VEME should do the same.

This would have caught the regression patched by blab/veme-2022@f14bbe6.

@jameshadfield jameshadfield added the enhancement New feature or request label Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant