Add comment why and how OptionalForks should be maintained.

status-im · Oct 31, 2024 · 16a8601 · 16a8601
1 parent 374cce3
commit 16a8601
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/beacon_chain/validator_client/common.nim b/beacon_chain/validator_client/common.nim
@@ -50,6 +50,11 @@ static: doAssert(high(ConsensusFork) == ConsensusFork.Electra,
           "Update OptionalForks constant!")
 const
   OptionalForks* = {ConsensusFork.Electra}
+    ## When a new ConsensusFork is added and before this fork is activated on
+    ## `mainnet`, it should be part of `OptionalForks`.
+    ## In this case, the client will ignore missing <FORKNAME>_VERSION
+    ## and <FORKNAME>_EPOCH constants from the data reported by BN via
+    ## `/eth/v1/config/spec` API call.
 
 type
   ServiceState* {.pure.} = enum