Comments fix

command/genesis/genesis.go

@@ -240,7 +240,7 @@ func setFlags(cmd *cobra.Command) {
 			&params.checkpointInterval,
 			checkpointIntervalFlag,
 			defaultCheckpointInterval,
-			"number of blocks after which a new checkpoint is submitted",
+			"checkpoint submission interval in blocks",
 		)
 
 		cmd.Flags().Uint64Var(

command/genesis/polybft_params.go

@@ -68,6 +68,8 @@ const (
 
 	ecdsaAddressLength = 40
 	blsKeyLength       = 256
+
+	proposalQuorumMax = uint64(100)
 )
 
 var (
@@ -168,9 +170,9 @@ func (p *genesisParams) generatePolyBftChainConfig(o command.OutputFormatter) er
 	}
 
 	proposalQuorum := p.proposalQuorum
-	if proposalQuorum > 100 {
+	if proposalQuorum > proposalQuorumMax {
 		// proposal can be from 0 to 100, so we sanitize the value
-		proposalQuorum = 100
+		proposalQuorum = proposalQuorumMax
 	}
 
 	polyBftConfig := &polybft.PolyBFTConfig{

consensus/polybft/governance_manager.go

@@ -7,23 +7,27 @@ import (
 	"math/big"
 	"time"
 
+	"github.com/hashicorp/go-hclog"
+	"github.com/umbracle/ethgo"
+	"github.com/umbracle/ethgo/abi"
+
 	"github.com/0xPolygon/polygon-edge/chain"
 	"github.com/0xPolygon/polygon-edge/consensus/polybft/contractsapi"
 	"github.com/0xPolygon/polygon-edge/crypto"
 	"github.com/0xPolygon/polygon-edge/forkmanager"
 	"github.com/0xPolygon/polygon-edge/helper/common"
 	"github.com/0xPolygon/polygon-edge/types"
-	"github.com/hashicorp/go-hclog"
-	"github.com/umbracle/ethgo"
-	"github.com/umbracle/ethgo/abi"
 )
 
 const (
 	oldRewardLookbackSize = uint64(2) // number of blocks to calculate commit epoch info from the previous epoch
 	newRewardLookbackSize = uint64(1)
 )
 
-var errUnknownGovernanceEvent = errors.New("unknown event from governance")
+var (
+	errUnknownGovernanceEvent = errors.New("unknown event from governance")
+	stringABIType             = abi.MustNewType("tuple(string)")
+)
 
 // isRewardDistributionBlock indicates if reward distribution transaction
 // should happen in given block
@@ -106,7 +110,6 @@ func newGovernanceManager(genesisConfig *PolyBFTConfig,
 
 	// cache all fork name hashes that we have in code
 	allForkNameHashes := map[types.Hash]string{}
-	stringABIType := abi.MustNewType("tuple(string)")
 
 	for name := range *chain.AllForksEnabled {
 		encoded, err := stringABIType.Encode([]interface{}{name})