Apply suggestions from code review

Co-authored-by: Rootul P <[email protected]>

Author: cmwaters

Diff for: pkg/inclusion/blob_share_commitment_rules.go

@@ -50,7 +50,7 @@ func BlobSharesUsedNonInteractiveDefaults(cursor, subtreeRootThreshold int, blob
 // fit in the square. The cursor is expected to be the index after the end of
 // the previous blob.
 //
-// See https://github.com/celestiaorg/go-square/blob/main/specs/src/specs/data_square_layout.md
+// See https://github.com/celestiaorg/celestia-app/blob/main/specs/src/specs/data_square_layout.md
 // for more information.
 func NextShareIndex(cursor, blobShareLen, subtreeRootThreshold int) int {
 	// Calculate the subtreewidth. This is the width of the first mountain in the

Diff for: pkg/shares/sparse_shares_test.go

@@ -83,7 +83,7 @@ func TestSparseShareSplitterCount(t *testing.T) {
 	}
 }
 
-// GenerateRandomBlobOfShareCount returns a blob that spans the given
+// generateRandomBlobOfShareCount returns a blob that spans the given
 // number of shares
 func generateRandomBlobOfShareCount(count int) *blob.Blob {
 	size := rawBlobSize(FirstSparseShareContentSize * count)

Diff for: pkg/square/builder.go

@@ -14,7 +14,7 @@ import (
 )
 
 type Builder struct {
-	// maxSquareSize is an upper bound on the total amount of shares that could fit in the biggest square
+	// maxSquareSize is the maximum number of rows (or columns) in the original data square
 	maxSquareSize int
 	// currentSize is an overestimate for the number of shares used by this builder.
 	currentSize int