fix staticcheck

autonat.go

@@ -443,5 +443,5 @@ func (s *StaticAutoNAT) PublicAddr() (ma.Multiaddr, error) {
 	if s.reachability != network.ReachabilityPublic {
 		return nil, errors.New("NAT status is not public")
 	}
-	return nil, errors.New("No available address")
+	return nil, errors.New("no available address")
 }

autonat_test.go

@@ -234,7 +234,7 @@ func TestAutoNATObservationRecording(t *testing.T) {
 	}
 
 	select {
-	case _ = <-s.Out():
+	case <-s.Out():
 		t.Fatal("not expecting a public reachability event")
 	default:
 		//expected

client.go

@@ -61,7 +61,7 @@ func (c *client) DialBack(ctx context.Context, p peer.ID) (ma.Multiaddr, error)
 	}
 
 	if res.GetType() != pb.Message_DIAL_RESPONSE {
-		return nil, fmt.Errorf("Unexpected response: %s", res.GetType().String())
+		return nil, fmt.Errorf("unexpected response: %s", res.GetType().String())
 	}
 
 	status := res.GetDialResponse().GetStatus()

dialpolicy_test.go

@@ -43,9 +43,7 @@ type mockL struct {
 }
 
 func (l *mockL) Accept() (transport.CapableConn, error) {
-	select {
-	case <-l.ctx.Done():
-	}
+	<-l.ctx.Done()
 	return nil, errors.New("expected in mocked test")
 }
 func (l *mockL) Close() error                   { return nil }

svc.go

@@ -28,7 +28,6 @@ type autoNATService struct {
 	config *config
 
 	// rate limiter
-	running    uint32
 	mx         sync.Mutex
 	reqs       map[peer.ID]int
 	globalReqs int
@@ -37,7 +36,7 @@ type autoNATService struct {
 // NewAutoNATService creates a new AutoNATService instance attached to a host
 func newAutoNATService(ctx context.Context, c *config) (*autoNATService, error) {
 	if c.dialer == nil {
-		return nil, errors.New("Cannot create NAT service without a network")
+		return nil, errors.New("cannot create NAT service without a network")
 	}
 
 	as := &autoNATService{