chore: remove redundant typeparams for Go 1.17 (#1331)

Author: alexandear

internal/typeparams/typeparams.go

@@ -6,24 +6,26 @@ import (
 	"go/ast"
 )
 
-// Enabled reports whether type parameters are enabled in the current build
-// environment.
-func Enabled() bool {
-	return enabled
-}
-
 // ReceiverType returns the named type of the method receiver, sans "*" and type
 // parameters, or "invalid-type" if fn.Recv is ill formed.
 func ReceiverType(fn *ast.FuncDecl) string {
 	e := fn.Recv.List[0].Type
 	if s, ok := e.(*ast.StarExpr); ok {
 		e = s.X
 	}
-	if enabled {
-		e = unpackIndexExpr(e)
-	}
+	e = unpackIndexExpr(e)
 	if id, ok := e.(*ast.Ident); ok {
 		return id.Name
 	}
 	return "invalid-type"
 }
+
+func unpackIndexExpr(e ast.Expr) ast.Expr {
+	switch e := e.(type) {
+	case *ast.IndexExpr:
+		return e.X
+	case *ast.IndexListExpr:
+		return e.X
+	}
+	return e
+}

internal/typeparams/typeparams_go117.go

@@ -3,15 +3,11 @@ package test
 import (
 	"testing"
 
-	"github.com/mgechev/revive/internal/typeparams"
 	"github.com/mgechev/revive/lint"
 	"github.com/mgechev/revive/rule"
 )
 
 func TestReceiverNamingTypeParams(t *testing.T) {
-	if !typeparams.Enabled() {
-		t.Skip("type parameters are not enabled in the current build environment")
-	}
 	testRule(t, "receiver_naming_issue_669", &rule.ReceiverNamingRule{})
 }