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

Refactor case counts workflow #102

Merged
merged 2 commits into from
Jun 7, 2024
Merged

Refactor case counts workflow #102

merged 2 commits into from
Jun 7, 2024

Conversation

joverlee521
Copy link
Contributor

@joverlee521 joverlee521 commented Jun 7, 2024

Description of proposed changes

Our push to use short-lived AWS credentials has finally pushed me to put this into a proper Snakemake workflow
that can be run by nextstrain build.

Related issue(s)

Related to https://github.com/nextstrain/private/issues/110

Checklist

The case counts scripts used csvtk before csvtk was officially
added to nextstrain/docker-base so I worked around this by just running
them directly in the GH Action workflow.

Our push to use short-lived AWS credentials has finally pushed me to
put this into a proper Snakemake workflow.
Refactored to use the shared `pathogen-repo-build` GH Action workflow
so that it can use the short-lived AWS credentials that are
automatically set up within the workflow.
@joverlee521 joverlee521 requested a review from a team June 7, 2024 17:55
@joverlee521 joverlee521 merged commit 56f7740 into main Jun 7, 2024
10 checks passed
@joverlee521 joverlee521 deleted the case-counts-workflow branch June 7, 2024 17:55
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.

1 participant