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: Switch CI to pathogen-repo-ci workflow #70

Merged
merged 2 commits into from
Jul 16, 2024
Merged

CI: Switch CI to pathogen-repo-ci workflow #70

merged 2 commits into from
Jul 16, 2024

Conversation

joverlee521
Copy link
Contributor

@joverlee521 joverlee521 commented Jul 16, 2024

Description of proposed changes

See nextstrain/.github#94 for context.

The ingest workflow runs in <1 minute so I figured it'd be nice
to add as an additional check in the GH Action CI workflow.
If the ingest workflow ever runs for too long, we should figure out a
way to subset the ingest data.

The ingest CI build config includes a bogus key because Snakemake requires the
config file to have at least one top level key.

Checklist

  • Checks pass

The ingest workflow current runs in <1 minute so I figured it'd be nice
to add as an additional check in the GH Action CI workflow.
If the ingest workflow ever runs for too long, we should figure out a
way to subset the ingest data.

The CI build config includes a bogus key because Snakemake requires the
config file to have at least one top level key.

Previous CI run without the bogus key ran into an error¹
```
snakemake.exceptions.WorkflowError: Config file must be given as JSON or YAML with keys at top level.
```

¹ <https://github.com/nextstrain/zika/actions/runs/9948628997/job/27483549394#step:9:17>
@joverlee521 joverlee521 merged commit e204dd4 into main Jul 16, 2024
4 checks passed
@joverlee521 joverlee521 deleted the update-ci branch July 16, 2024 16:44
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.

2 participants