nits

Author: dhrubabasu

vms/platformvm/block/builder/builder.go

@@ -130,7 +130,7 @@ func (b *builder) buildBlock() (block.Block, error) {
 
 	timestamp, timeWasCapped, err := txexecutor.NextBlockTime(preferredState, b.txExecutorBackend.Clk)
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("could not calculate next staker change time: %w", err)
 	}
 
 	return buildBlock(

vms/platformvm/state/state.go

@@ -1447,7 +1447,7 @@ func (s *state) loadCurrentValidators() error {
 		}
 		tx, _, err := s.GetTx(txID)
 		if err != nil {
-			return fmt.Errorf("failed loading validator transaction txID %v, %w", txID, err)
+			return err
 		}
 
 		stakerTx, ok := tx.Unsigned.(txs.Staker)

vms/platformvm/txs/executor/standard_tx_executor.go

@@ -533,7 +533,7 @@ func (e *StandardTxExecutor) BaseTx(tx *txs.BaseTx) error {
 	return nil
 }
 
-// putStaker creates the staker and adds it to state.
+// Creates the staker as defined in [stakerTx] and adds it to [e.State].
 func (e *StandardTxExecutor) putStaker(stakerTx txs.Staker) error {
 	txID := e.Tx.ID()
 	staker, err := state.NewPendingStaker(txID, stakerTx)

vms/platformvm/txs/executor/tx_mempool_verifier.go

@@ -5,7 +5,6 @@ package executor
 
 import (
 	"errors"
-	"fmt"
 	"time"
 
 	"github.com/ava-labs/avalanchego/ids"
@@ -136,7 +135,7 @@ func NextBlockTime(state state.Chain, clk *mockable.Clock) (time.Time, bool, err
 
 	nextStakerChangeTime, err := GetNextStakerChangeTime(state)
 	if err != nil {
-		return time.Time{}, false, fmt.Errorf("could not calculate next staker change time: %w", err)
+		return time.Time{}, false, err
 	}
 
 	// timeWasCapped means that [timestamp] was reduced to