Some minor fixes (#7082)

* remove duplicated check

* remove redundant return

* fix build error

* format process_withdrawal_request

Author: nethoxa

beacon_chain/spec/state_transition_block.nim

@@ -418,9 +418,6 @@ proc check_voluntary_exit*(
     return err("Exit: not in validator set long enough")
 
   when typeof(state).kind >= ConsensusFork.Electra:
-    if voluntary_exit.validator_index >= state.validators.lenu64:
-      return err("Exit: validator index out of range")
-
     # Only exit validator if it has no pending withdrawals in the queue
     if not (get_pending_balance_to_withdraw(
         state, voluntary_exit.validator_index.ValidatorIndex) == 0.Gwei):
@@ -531,9 +528,7 @@ func process_withdrawal_request*(
   if is_full_exit_request:
     # Only exit validator if it has no pending withdrawals in the queue
     if pending_balance_to_withdraw == 0.Gwei:
-      if initiate_validator_exit(cfg, state, index, default(ExitQueueInfo),
-          cache).isErr():
-        return
+      discard initiate_validator_exit(cfg, state, index, ExitQueueInfo(), cache)
     return
 
   let