ChainSafe / ChainSafe/lodestar
Review usage of initializeBeaconStateFromEth1
Open
scope-testing
- Dominant language
- TypeScript
- Stars
- 1.4k
- Forks
- 483
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 156
Description
Reference: https://github.com/ethereum/consensus-specs/pull/4017
As described by @ensi321 , `initialize_beacon_state_from_eth1` is being removed from spec. Review usage in Lodestar.
Currently is used in `spec/presets/genesis.test.ts` and `initDevState()`
Contributor guide
Research direction
Start by reviewing consensus-specs PR 4017 and the two Lodestar usages named in the issue: spec/presets/genesis.test.ts and initDevState(). Trace how initialize_beacon_state_from_eth1 is used in each location and compare them with the updated spec. Done means the remaining usage is updated or removed consistently with the spec.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- blockchain
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100