-
Notifications
You must be signed in to change notification settings - Fork 714
attempt to use local actions #10503
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
base: master
Are you sure you want to change the base?
attempt to use local actions #10503
Conversation
8565a7c
to
501bde6
Compare
b9cd73b
to
1215808
Compare
okay, took a bit of fighting only to discover that a couple of typoes led to empty strings instead of useful errors, but I've merged the validate and bootstrap lists (see comments in |
Next push will have a preliminary changelog. Would still like feedback on documentation, but I suspect we need to get 3.16 out of the way first. |
8a4bd44
to
99fb64f
Compare
I ended up switching the changelog to a Possible change I'm considering: someone has a Other considerations:
|
99fb64f
to
a317225
Compare
Bloody caches… |
…right, and now it runs. Still, will need to switch to artifacts to avoid stuff randomly failing because GitHub ran their cache cleanup. |
Welp. I researched how to save just the testing artifacts, got suspicious about |
Extremely experimental, and with all the limitations and restrictions I keep finding in GitHub Actions it'll probably fail in the messiest way it can. At present this is incomplete but sufficient to see if this has any chance of working to begin with. If it somehow does, I'll look into abstracting out the other sub-jobs, then making an overnight validate for Tier 2 platforms and probably a prerelease job (which would fix the recently revealed problem where if there is no need to rebase on merge, no prerelease is made).
a317225
to
3c3a3d2
Compare
This PR redesigns our PR CI:
config.yml
file rather than scattering them throughout the codebaseIt also paves the way for things like
(Note that, as part of the second point above, this rewrite doesn't currently do prereleases at all. I can re-add that until said dedicated job is deployed.)
More details are in
.github/README.md
, which additionally details various other parts of our CI infrastructure.Template B: This PR does not modify behaviour or interface
E.g. the PR only touches documentation or tests, does refactorings, etc.
Include the following checklist in your PR:
Is this a PR that fixes CI? If so, it will need to be backported to older cabal release branches (ask maintainers for directions).(No point in backporting, as even the LTS prerelease part has to live onmaster
.)