Closed jeluard closed 3 weeks ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 62.76%. Comparing base (
966f3ac
) to head (8d7bc17
). Report is 10 commits behind head on unstable.
✔️ no performance regression detected
🚀🚀 Significant benchmark improvement detected
Benchmark suite | Current: 07419b767cca4f1c1a9f546e62e2edf1fa4066b5 | Previous: 3be656bb2279c9ed5ab8d3117e368d02b21c5046 | Ratio |
---|---|---|---|
forkChoice updateHead vc 100000 bc 64 eq 0 | 385.18 us/op | 1.2012 ms/op | 0.32 |
by benchmarkbot/action
Noticed there are plenty of links in our repo itself which lead to a "Page Not Found"
need to make sure to update those if we restructure docs
forgot to mention this one on the call, it would be nice to get this merged for 1.20
Approving everything up to my commit. Fixed some minor details and ready to go unless someone has strong opposition to this current structure.
Minor thing I just noticed
subsections need some better titles
also "Lodestar Eth Consensus Lightclient Prover" is a ridiculous name :joy:
I'll be fixing up specific and outdated content in a new PR. Will rename that readme title there 😅
Motivation
Make sure all relevant files are available in docs. Also improve collapsed section.