-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add nextstrain-automation build-configs to phylogenetic worklfow #37
Conversation
Copied commit from zika: nextstrain/zika@810e1d2 As duplicating nextstrain/zika#51
Add config and rules for Nextstrain automation to deploy the Dengue dataset Copied commit from zika: nextstrain/zika@810e1d2 As duplicating nextstrain/zika#51
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for following up here, changes look good by inspection!
@@ -0,0 +1,15 @@ | |||
""" | |||
This part of the workflow handles automatic deployments of the zika build. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This part of the workflow handles automatic deployments of the zika build. | |
This part of the workflow handles automatic deployments of the dengue build. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@victorlin @joverlee521 , could we generalize this and just say "pathogen" build?
This way there's less to trip-over when applying these rules to other pathogens of interest.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, submitted as a hotfix: b2c3655
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea, +1 for less pathogen-specific stuff.
Pointed out post merge by commit suggestion in PR #37: #37 (comment)
Description of proposed changes
Adds a custom config and rules for deploying the final Auspice dataset of the phylogenetic workflow.
From slack, had permission to duplicate changes from nextstrain/zika#51
Related issue(s)
Checklist