Modernise the Cygwin CI setup#85
Merged
Merged
Conversation
wilfwilson
force-pushed
the
ci
branch
2 times, most recently
from
2713a10 to
744425a
Compare
wilfwilson
force-pushed
the
ci
branch
2 times, most recently
from
af0985a to
2f73ddb
Compare
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This changes the CI for the Cygwin job to be almost identical to the CI for the Ubuntu jobs. This is now possible because of some improvements to the
gap-actions/setup-cygwinaction and the rest of thegap-actionsstuff.I haven't actually merged the jobs, as I normally would and as I suggested in #82, because the Cygwin job is really slow (6 hours?), and so we don't want it to run whenever we run the Ubuntu jobs. So having it as a separate Workflow means that it can be triggered in different circumstances.
These circumstances now are manually, on schedule once a week, and I've newly added on pushes to
master, which is something that I think we do infrequently enough that it's worth the cost.I've run the Cygwin job on my fork, you can see it progressing at https://github.com/wilfwilson/vole/actions/workflows/cygwin.yml
Resolves #82.