Add lint check for duplicated imports (#439)

* Enable duplicated imports check

* Remove duplicated import

exporter/collector/logs.go

@@ -27,7 +27,6 @@ import (
 	"cloud.google.com/go/logging"
 	loggingv2 "cloud.google.com/go/logging/apiv2"
 	"google.golang.org/genproto/googleapis/api/monitoredres"
-	monitoredrespb "google.golang.org/genproto/googleapis/api/monitoredres"
 	logpb "google.golang.org/genproto/googleapis/logging/v2"
 	"google.golang.org/protobuf/proto"
 
@@ -239,7 +238,7 @@ func (l logMapper) logEntryToInternal(
 	entry logging.Entry,
 	logName string,
 	projectID string,
-	mr *monitoredrespb.MonitoredResource,
+	mr *monitoredres.MonitoredResource,
 	splits int,
 	splitIndex int,
 ) (*logpb.LogEntry, error) {

exporter/metric/metric.go

@@ -35,7 +35,6 @@ import (
 
 	monitoring "cloud.google.com/go/monitoring/apiv3/v2"
 	"google.golang.org/api/option"
-	apioption "google.golang.org/api/option"
 	"google.golang.org/genproto/googleapis/api/distribution"
 	googlemetricpb "google.golang.org/genproto/googleapis/api/metric"
 	monitoredrespb "google.golang.org/genproto/googleapis/api/monitoredres"
@@ -129,7 +128,7 @@ func newMetricExporter(o *options) (*metricExporter, error) {
 		return nil, errBlankProjectID
 	}
 
-	clientOpts := append([]apioption.ClientOption{option.WithUserAgent(userAgent)}, o.monitoringClientOptions...)
+	clientOpts := append([]option.ClientOption{option.WithUserAgent(userAgent)}, o.monitoringClientOptions...)
 	ctx := o.context
 	if ctx == nil {
 		ctx = context.Background()

exporter/metric/metric_test.go

@@ -24,7 +24,6 @@ import (
 
 	"go.opentelemetry.io/otel/attribute"
 	"go.opentelemetry.io/otel/sdk/instrumentation"
-	"go.opentelemetry.io/otel/sdk/metric/controller/basic"
 	controller "go.opentelemetry.io/otel/sdk/metric/controller/basic"
 	"go.opentelemetry.io/otel/sdk/metric/export"
 	"go.opentelemetry.io/otel/sdk/metric/export/aggregation"
@@ -94,7 +93,7 @@ func TestExportCounter(t *testing.T) {
 
 	clientOpt := option.WithGRPCConn(cloudMock.ClientConn())
 
-	resOpt := basic.WithResource(
+	resOpt := controller.WithResource(
 		resource.NewWithAttributes(
 			semconv.SchemaURL,
 			attribute.String("test_id", "abc123"),
@@ -130,7 +129,7 @@ func TestExportHistogram(t *testing.T) {
 
 	clientOpt := option.WithGRPCConn(cloudMock.ClientConn())
 
-	resOpt := basic.WithResource(
+	resOpt := controller.WithResource(
 		resource.NewWithAttributes(
 			semconv.SchemaURL,
 			attribute.String("test_id", "abc123"),
@@ -199,7 +198,7 @@ func TestRecordToMpb(t *testing.T) {
 
 	clientOpt := option.WithGRPCConn(cloudMock.ClientConn())
 
-	resOpt := basic.WithResource(
+	resOpt := controller.WithResource(
 		resource.NewWithAttributes(
 			semconv.SchemaURL,
 			attribute.String("test_id", "abc123"),

golangci.yml

@@ -106,6 +106,9 @@ linters-settings:
     - metres
     - kilometre
     - kilometres
+  revive:
+    rules:
+      - name: duplicated-imports
 
 linters:
   disable-all: true