sigp / enr

Ethereum Node Record
MIT License
61 stars 29 forks source link

fix: compilation with `default-features = false` #74

Closed DaniPopes closed 5 months ago

DaniPopes commented 6 months ago

Fix compilation with default-features = false, meaning without the k256 feature, also testing it in CI.

I took the opportunity to also improve the CI with more checks, I adapted it from what we have at alloy-rs/core.

DaniPopes commented 5 months ago

Yes, these are intentional.

Running time went from 3m cached to 40s uncached (https://github.com/sigp/enr/actions/runs/8618027272). It technically does use more resources and has more billable time but running it is free so it ultimately doesn't matter.