Migrate CI to GitHub Actions #16
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
If this is acceptable to folks, then I would also plan to refactor it out into https://github.com/nextstrain/.github as a starter workflow + reusable workflow pair that we can then apply (and maintain) across the board to all/most of our pathogen repos. |
j23414
approved these changes
Apr 1, 2022
victorlin
requested changes
Apr 1, 2022
We've been slowly moving everything off Travis CI in favor of centralizing on GitHub. More acutely motivated here by some Travis-specific errors, previously worked around by <#14>. Reverts back to a simple pip install instead of conda/mamba setup as much of it was unnecessary. See discussion in <nextstrain/measles#3 (comment)>.
victorlin
approved these changes
Apr 1, 2022
tsibley
added a commit
to nextstrain/.github
that referenced
this pull request
Apr 1, 2022
The goal is to establish a common setup for CI for all or at least most of our pathogen build repos. Includes a reusable workflow¹ which can be called/invoked by our workflows in other repos and a "starter" workflow template² that performs such a call, for copying into the pathogen repo (either manually or via the web UI which supports these starter workflows). These are both basic for now but appropriate for our "simple" pathogen builds like zika and measles. We can add workflow parameters and conditionals as necessary in the future to support CI for more complex builds, like ncov. (It won't take much!) Related to <nextstrain/zika#16>. ¹ https://docs.github.com/en/actions/using-workflows/reusing-workflows ² https://docs.github.com/en/actions/using-workflows/creating-starter-workflows-for-your-organization
tsibley
added a commit
to nextstrain/.github
that referenced
this pull request
Apr 1, 2022
The goal is to establish a common setup for CI for all or at least most of our pathogen build repos. Includes a reusable workflow¹ which can be called/invoked by our workflows in other repos and a "starter" workflow template² that performs such a call, for copying into the pathogen repo (either manually or via the web UI which supports these starter workflows). These are both basic for now but appropriate for our "simple" pathogen builds like zika and measles. We can add workflow parameters and conditionals as necessary in the future to support CI for more complex builds, like ncov. (It won't take much!) Related to <nextstrain/zika#16>. ¹ https://docs.github.com/en/actions/using-workflows/reusing-workflows ² https://docs.github.com/en/actions/using-workflows/creating-starter-workflows-for-your-organization
This was referenced Apr 1, 2022
tsibley
added a commit
to nextstrain/.github
that referenced
this pull request
Apr 1, 2022
The goal is to establish a common setup for CI for all or at least most of our pathogen build repos. Includes a reusable workflow¹ which can be called/invoked by our workflows in other repos and a "starter" workflow template² that performs such a call, for copying into the pathogen repo (either manually or via the web UI which supports these starter workflows). These are both basic for now but appropriate for our "simple" pathogen builds like zika and measles. We can add workflow parameters and conditionals as necessary in the future to support CI for more complex builds, like ncov. (It won't take much!) Related to <nextstrain/zika#16>. ¹ https://docs.github.com/en/actions/using-workflows/reusing-workflows ² https://docs.github.com/en/actions/using-workflows/creating-starter-workflows-for-your-organization
tsibley
added a commit
to nextstrain/.github
that referenced
this pull request
Apr 1, 2022
The goal is to establish a common setup for CI for all or at least most of our pathogen build repos. Includes a reusable workflow¹ which can be called/invoked by our workflows in other repos and a "starter" workflow template² that performs such a call, for copying into the pathogen repo (either manually or via the web UI which supports these starter workflows). These are both basic for now but appropriate for our "simple" pathogen builds like zika and measles. We can add workflow parameters and conditionals as necessary in the future to support CI for more complex builds, like ncov. (It won't take much!) Related to <nextstrain/zika#16>. ¹ https://docs.github.com/en/actions/using-workflows/reusing-workflows ² https://docs.github.com/en/actions/using-workflows/creating-starter-workflows-for-your-organization
tsibley
added a commit
to nextstrain/measles
that referenced
this pull request
Apr 1, 2022
This was referenced Apr 4, 2022
tsibley
pushed a commit
to nextstrain/avian-flu
that referenced
this pull request
Apr 4, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We've been slowly moving everything off Travis CI in favor of
centralizing on GitHub. More acutely motivated here by some
Travis-specific errors, previously worked around by
#14.
Reverts back to a simple pip install instead of conda/mamba setup as
much of it was unnecessary. See discussion in
nextstrain/measles#3 (comment).