Closed nazarhussain closed 1 week ago
Attention: Patch coverage is 80.00000%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 62.54%. Comparing base (
802a875
) to head (3675a0a
).
✔️ no performance regression detected
by benchmarkbot/action
Motivation
We don't want to hide or shadow an actual consensus type name. And as defined in
params
package the blinded is not a defined fork group, so rather tend to use actual group name everywhere which is execution.Description
Based on discussion we dont want to hide/shadow an actual type. The blinded types implementation renames types, e.g.
BlindedBeaconBlock
toBeaconBlock
. Which is another consensus type. No such pattern used elsewhere for other fork groups.Blinded itself is not defined as a fork group in
params
package, so keeping that implementation requires an extra effort.Steps to test or reproduce