ci: improvements - Githubissues

ethereum / go-verkle

A go implementation of Verkle trees

The Unlicense

209 stars 63 forks source link

ci: improvements #451

Closed jsign closed 2 weeks ago

jsign commented 2 weeks ago

This PR improves the CI configuration to:

Remove workflows that now have native support in GH through Settings.
Stop using self-hosted for build/lint/test. It isn't properly configured and took ~30min to run for no good reason (i.e: has an ever growing Go cache restoring).
The new build/test covers three OS.

jsign commented 2 weeks ago

Regarding selfhosted, we are currently looking into fixing this problem, which needs to be solved for geth as well. So I don't think there is a need to move away from it, especially since ubuntu-latest is not powerful enough to run all the geth tests.

Why is ubuntu-latest power related to go-verkle? geth tests can still be run with self-hosted if that's useful. What's the benefit of running self-hosted in this repo?

gballet commented 2 weeks ago

I guess. Ok, let's do that.