Apply suggestions from code review

Co-authored-by: M. J. Fromberger <michael.j.fromberger@gmail.com>

internal/p2p/metrics.go

@@ -28,7 +28,6 @@ var (
 type Metrics struct {
 	// Number of peers connected.
 	PeersConnected metrics.Gauge
-
 	// Nomber of peers in the peer store database.
 	PeersStored metrics.Gauge
 	// Number of inactive peers stored.

internal/p2p/peermanager_scoring_test.go

@@ -115,14 +115,9 @@ func makeMockPeerStore(t *testing.T, peers ...peerInfo) *peerStore {
 func TestPeerRanking(t *testing.T) {
 	t.Run("InactiveSecond", func(t *testing.T) {
 		store := makeMockPeerStore(t,
-			peerInfo{
-				ID:       "second",
-				Inactive: true,
-			},
-			peerInfo{
-				ID:       "first",
-				Inactive: false,
-			})
+			peerInfo{ID: "second", Inactive: true},
+			peerInfo{ID: "first", Inactive: false},
+		)
 
 		ranked := store.Ranked()
 		if len(ranked) != 2 {

internal/p2p/router.go

@@ -444,8 +444,8 @@ func (r *Router) filterPeersID(ctx context.Context, id types.NodeID) error {
 func (r *Router) dialSleep(ctx context.Context) {
 	if r.options.DialSleep == nil {
 		const (
-			maxDialerInterval = 500
-			minDialerInterval = 100
+			maxDialerInterval = 500  // ms
+			minDialerInterval = 100  // ms
 		)
 
 		// nolint:gosec // G404: Use of weak random number generator