status-im / nimbus-eth2

Nim implementation of the Ethereum Beacon Chain
https://nimbus.guide
Other
523 stars 227 forks source link

extend light client protocol for Electra #6375

Closed etan-status closed 3 months ago

etan-status commented 3 months ago

Add missing Electra support for light client protocol:

Tested against PR consensus-spec-tests, the test runner automatically picks up the new tests once available.

github-actions[bot] commented 3 months ago

Unit Test Results

         9 files  ±  0    1 334 suites  +6   31m 32s :stopwatch: + 5m 3s   4 998 tests ±  0    4 650 :heavy_check_mark: ±  0  348 :zzz: ±0  0 :x: ±0  20 874 runs  +24  20 470 :heavy_check_mark: +24  404 :zzz: ±0  0 :x: ±0 

Results for commit a5c8bae9. ± Comparison against base commit 3da85e59.

:recycle: This comment has been updated with latest results.

tersec commented 3 months ago

Also for all the "... works in Nim 1.6 ..." etc comments, as of the last couple days, nimbus-eth2 uses Nim 2.0.6, so that's the relevant version to check. No need to find/update all of them, though, apparently the upstream issue isn't fixed, but at least new comments going in should reference 2.0 or 2.0.6 specifically.