Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
kdeme pushed 4 commits to historical-summaries-endpoint status-im/nimbus-eth2
kdeme pushed 1 commit to historical-summaries-endpoint status-im/nimbus-eth2
- Move endpoint to /nimbus/v1/debug/ namespace for now e3d435b
kdeme created a comment on a pull request on status-im/nimbus-eth2
> One consideration for me is that if this does get standardized, we don't end up with both a pre-standardized and standardized version which needs to be supported. One approach could be that I ...
tersec created a review comment on a pull request on status-im/nimbus-eth2
This is causing a compilation error: ``` 2024-10-24T07:56:38.1576563Z /github-runner/github-runner-node-02/workspace/nimbus-eth2/nimbus-eth2/beacon_chain/validators/validator_duties.nim(43, 22) E...
tersec created a comment on a pull request on status-im/nimbus-eth2
One consideration for me is that if this does get standardized, we don't end up with both a pre-standardized and standardized version which needs to be supported.
tersec pushed 1 commit to unstable status-im/nimbus-eth2
- unhide ObservableStores and hide ImplicitTemplateRedefinition warnings (#6678) 8a6eab7
tersec closed a pull request on status-im/nimbus-eth2
unhide ObservableStores and hide ImplicitTemplateRedefinition warnings
`ObservableStores` warnings appear to have vanished.
kdeme pushed 1 commit to historical-summaries-endpoint status-im/nimbus-eth2
- Adress review comments: Correct errors + asyncraises 099ed00
github-actions[bot] created a comment on a pull request on status-im/nimbus-eth2
## Unit Test Results 12 files ±0 1 810 suites ±0 54m 43s [:stopwatch:](https://github.com/EnricoMi/publish-unit-test-result-action/blob/v1.20/README.md#the-symbols "duration of all te...
tersec created a review comment on a pull request on status-im/nimbus-eth2
Also, it's triggering a bunch of errors: ``` 2024-10-24T07:54:41.4400980Z /github-runner/github-runner-node-01/workspace/nimbus-eth2/nimbus-eth2/beacon_chain/consensus_object_pools/blockchain_dag...
tersec created a review comment on a pull request on status-im/nimbus-eth2
we don't need to define these yet -- can just keep `onUpgradeToFulu` `nil` and they'll never be used
tersec created a review comment on a pull request on status-im/nimbus-eth2
`var (electra.BeaconState | fulu.BeaconState)`
tersec created a review comment on a pull request on status-im/nimbus-eth2
see elsewhere, `var (electra.BeaconState | fulu.BeaconState)`
tersec created a review comment on a pull request on status-im/nimbus-eth2
can use `state: var (electra.BeaconState | fulu.BeaconState)` rather than this `T` indirection
tersec created a review comment on a pull request on status-im/nimbus-eth2
See usual point about attestations not known to be yet again changing in Fulu, can just use Electra attestions (functions, data types, logging, etc)
tersec created a review comment on a pull request on status-im/nimbus-eth2
`SomeIndexedAttetation` and `SomeAttesterSlashing` shouldn't be changing (as far as is known) in Fulu, no need to copy these two `shortLog` functions