chore: improve code comments clarity (#2943)

<!--
Please read and fill out this form before submitting your PR.

Please make sure you have reviewed our contributors guide before
submitting your
first PR.

NOTE: PR titles should follow semantic commits:
https://www.conventionalcommits.org/en/v1.0.0/
-->

## Overview

<!-- 
Please provide an explanation of the PR, including the appropriate
context,
background, goal, and rationale. If there is an issue with this
information,
please provide a tl;dr and link the issue. 

Ex: Closes #<issue number>
-->

improve code comments clarity

Signed-off-by: stringscut <[email protected]>

Author: stringscut

block/internal/cache/pending_headers.go

@@ -17,7 +17,7 @@ import (
 // - DA submission of multiple headers is atomic - it's impossible to submit only part of a batch
 //
 // lastSubmittedHeaderHeight is updated only after receiving confirmation from DA.
-// Worst case scenario is when headers was successfully submitted to DA, but confirmation was not received (e.g. node was
+// Worst case scenario is when headers were successfully submitted to DA, but confirmation was not received (e.g. node was
 // restarted, networking issue occurred). In this case headers are re-submitted to DA (it's extra cost).
 // evolve is able to skip duplicate headers so this shouldn't affect full nodes.
 // TODO(tzdybal): we shouldn't try to push all pending headers at once; this should depend on max blob size

block/internal/syncing/da_retriever.go

@@ -345,7 +345,7 @@ func createEmptyDataForHeader(ctx context.Context, header *types.SignedHeader) *
 			ChainID:      header.ChainID(),
 			Height:       header.Height(),
 			Time:         header.BaseHeader.Time,
-			LastDataHash: nil, // LastDataHash must be filled in the syncer, as it is not available here, block n-1 has not been processed yet.
+			LastDataHash: nil, // LastDataHash must be filled in the syncer, as it is not available here since block n-1 has not been processed yet.
 		},
 	}
 }

block/internal/syncing/da_retriever_test.go

@@ -182,7 +182,7 @@ func TestDARetriever_ProcessBlobs_HeaderOnly_EmptyDataExpected(t *testing.T) {
 	assert.True(t, ok)
 	assert.Equal(t, uint64(88), hHeight)
 
-	// empty data is not marked as data included (the submitter components does handle the empty data case)
+	// empty data is not marked as data included (the submitter component does handle the empty data case)
 	_, ok = r.cache.GetDataDAIncluded(events[0].Data.DACommitment().String())
 	assert.False(t, ok)
 }

pkg/genesis/genesis.go

@@ -18,7 +18,7 @@ type Genesis struct {
 	// DAStartHeight corresponds to the height at which the first DA header/data has been published.
 	// This value is meant to be updated after genesis and shared to all syncing nodes for speeding up syncing via DA.
 	DAStartHeight uint64 `json:"da_start_height"`
-	// DaEpochForcedInclusion corresponds to the amount of DA blocks are considered an epochs
+	// DaEpochForcedInclusion corresponds to the amount of DA blocks are considered an epoch
 	// When forced inclusion is enabled, the epoch size determines at what frequency the forced included transactions are executed by the application.
 	DAEpochForcedInclusion uint64 `json:"da_epoch_forced_inclusion"`
 }

sequencers/based/sequencer.go

@@ -134,7 +134,7 @@ func (s *BasedSequencer) GetNextBatch(ctx context.Context, req coresequencer.Get
 	}
 
 	// Calculate timestamp based on remaining transactions after this batch
-	// timestamp correspond to the last block time of a DA epoch, based on the remaining transactions to be executed
+	// timestamp corresponds to the last block time of a DA epoch, based on the remaining transactions to be executed
 	// this is done in order to handle the case where a DA epoch must fit in multiple blocks
 	remainingTxs := uint64(len(s.currentBatchTxs)) - s.checkpoint.TxIndex
 	timestamp := s.currentDAEndTime.Add(-time.Duration(remainingTxs) * time.Millisecond)