Skip to content

Update workflows for nextstrain run#93

Merged
joverlee521 merged 1 commit intomainfrom
nextstrain-run-support
Nov 4, 2025
Merged

Update workflows for nextstrain run#93
joverlee521 merged 1 commit intomainfrom
nextstrain-run-support

Conversation

@joverlee521
Copy link
Copy Markdown
Contributor

@joverlee521 joverlee521 commented Sep 23, 2025

Description of proposed changes

Workflow changes to support running the workflows with the nextstrain run command.

Declares compatibility for nextstrain run in the nextstrain-pathogen.yaml for the ingest workflow. Does not declare compatibility for phylo and nextclade workflows because they are incomplete in this guide.

Related issue(s)

nextstrain/public#1

Checklist

  • Checks pass
  • Update changelog

Base automatically changed from user-data-pattern to main September 24, 2025 22:03
@joverlee521 joverlee521 force-pushed the nextstrain-run-support branch from ea408f5 to ecca55d Compare October 31, 2025 19:52
@joverlee521
Copy link
Copy Markdown
Contributor Author

Rebased to fix merge conflict. I'll merge this on Monday if there are no comments.

Comment thread nextstrain-pathogen.yaml
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit messages are outdated:

  • 1deaceb – workflow specific compatibility is already supported
  • ecca55d – "Add support for nextstrain run" (386f5d9) references an old dangling commit

I think it'd be fine to squash both into a single commit.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, thanks for flagging. Will squash them.

Workflow changes to support running the workflows with the
`nextstrain run` command.

Declares compatibility for `nextstrain run` for the ingest workflow, but
does not declare compatibility for phylo and nextclade workflows because
they are incomplete in this guide.

Co-authored-by: Thomas Sibley <tsibley@fredhutch.org>
@joverlee521 joverlee521 force-pushed the nextstrain-run-support branch from ecca55d to d42a300 Compare November 4, 2025 00:49
@joverlee521 joverlee521 merged commit e42caed into main Nov 4, 2025
1 check passed
@joverlee521 joverlee521 deleted the nextstrain-run-support branch November 4, 2025 00:50
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