-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Better ci infrastructure #5445
Better ci infrastructure #5445
Conversation
No one looks at this output and it doesn't fail or notify us if performance goes bad so there is really very little gain from the extra time it takes.
Has built in caching so we might get faster builds for incremental builds where dependencies hasn't changed.
9c58ec2
to
5e2aaf5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool
Thanks. Feel free to update required workflows and merge when you are ready :) |
use workaround until actions/setup-go#450 is implemented.
I added a little change so we can load the version from |
Perfect thanks, I think that is the settings updated |
Great. Just an approval needed before it can be merged :) |
Description:
Various changes to make it easier to maintain (or rather have it maintain itself) and also some cleanups plus simplifications.
actions/setup-go
which is better maintained and has built in dependency caching infrastructure (a fresh build aftergo.mod
changes is likely slower but we avoid downloading dependencies for all else).go.mod
automatically using workaround until Use version from module and another version in matrix testing actions/setup-go#450 is implemented.Checklist: