search

status-im / nimbus-eth2

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

Bump nim-eth module and add async raises for eth2 discovery #6344

Closed kdeme closed 3 months ago

kdeme commented 3 months ago

The nim-eth bump has more changes than just the async raises:

c3f9160 Add async raises annotations for uTP code (status-im/nim-eth#692) 3d66c5b Fix ImplicitDefaultValue warnings + replace some obsolete inits (status-im/nim-eth#697) 50e71b2 Add async raises annotations for discv5 (status-im/nim-eth#690) c9b545b No sink (status-im/nim-eth#695) 5599435 ethblock: txs -> transactions (status-im/nim-eth#694) b9b522f keccak doesn't need init (status-im/nim-eth#693) c02e050 avoid burnMem for keccakHash (status-im/nim-eth#691) d935c0d rlp: avoid or make faster seq copies (status-im/nim-eth#689) 68bd675 faster hash for Address (status-im/nim-eth#688) 0e83cfd results: use standalone repo (status-im/nim-eth#687) 50b3d01 Revert missing hexary trie node work (status-im/nim-eth#686) 2587988 eth_types: remove unused ref types (status-im/nim-eth#685) bb5cb6a rlp: refresh code (status-im/nim-eth#683) 0addffc binary tries: remove (status-im/nim-eth#684)

The affecting ones for nimbus-eth2 would be status-im/nim-eth#692, status-im/nim-eth#689 and status-im/nim-eth#683

github-actions[bot] commented 3 months ago

Unit Test Results

         9 files  ±0    1 325 suites  ±0   33m 4s :stopwatch: - 1m 12s   4 986 tests ±0    4 638 :heavy_check_mark: ±0  348 :zzz: ±0  0 :x: ±0  20 814 runs  ±0  20 410 :heavy_check_mark: ±0  404 :zzz: ±0  0 :x: ±0 

Results for commit dd9aca09. ± Comparison against base commit 77d92652.