Standardize single import formats (ava-labs#1466)

Author: dhrubabasu

api/health/metrics.go

@@ -3,9 +3,7 @@
 
 package health
 
-import (
-	"github.com/prometheus/client_golang/prometheus"
-)
+import "github.com/prometheus/client_golang/prometheus"
 
 type metrics struct {
 	// failingChecks keeps track of the number of check failing

api/health/result.go

@@ -3,9 +3,7 @@
 
 package health
 
-import (
-	"time"
-)
+import "time"
 
 // notYetRunResult is the result that is returned when a HealthCheck hasn't been
 // run yet.

api/server/middleware_handler.go

@@ -3,9 +3,7 @@
 
 package server
 
-import (
-	"net/http"
-)
+import "net/http"
 
 type middlewareHandler struct {
 	before, after func()

api/server/wrapper.go

@@ -3,9 +3,7 @@
 
 package server
 
-import (
-	"net/http"
-)
+import "net/http"
 
 type Wrapper interface {
 	// WrapHandler wraps an http.Handler.

chains/atomic/writer.go

@@ -3,9 +3,7 @@
 
 package atomic
 
-import (
-	"github.com/ava-labs/avalanchego/database"
-)
+import "github.com/ava-labs/avalanchego/database"
 
 // WriteAll writes all of the batches to the underlying database of baseBatch.
 // Assumes all batches have the same underlying database.

ids/test_generator.go

@@ -3,9 +3,7 @@
 
 package ids
 
-import (
-	"sync/atomic"
-)
+import "sync/atomic"
 
 var offset = uint64(0)
 

network/conn_test.go

@@ -3,9 +3,7 @@
 
 package network
 
-import (
-	"net"
-)
+import "net"
 
 var _ net.Conn = (*testConn)(nil)
 

scripts/lint.sh

@@ -21,7 +21,7 @@ fi
 # by default, "./scripts/lint.sh" runs all lint tests
 # to run only "license_header" test
 # TESTS='license_header' ./scripts/lint.sh
-TESTS=${TESTS:-"golangci_lint license_header require_error_is_no_funcs_as_params"}
+TESTS=${TESTS:-"golangci_lint license_header require_error_is_no_funcs_as_params single_import"}
 
 function test_golangci_lint {
   go install -v github.com/golangci/golangci-lint/cmd/golangci-lint@v1.51.2
@@ -51,6 +51,13 @@ function test_license_header {
   "${files[@]}"
 }
 
+function test_single_import {
+  if grep -R -zo -P 'import \(\n\t".*"\n\)' .; then
+    echo ""
+    return 1
+  fi
+}
+
 function test_require_error_is_no_funcs_as_params {
   if grep -R -zo -P 'require.ErrorIs\(.+?\)[^\n]*\)\n' .; then
     echo ""

snow/consensus/metrics/height.go

@@ -3,9 +3,7 @@
 
 package metrics
 
-import (
-	"github.com/prometheus/client_golang/prometheus"
-)
+import "github.com/prometheus/client_golang/prometheus"
 
 var _ Height = (*height)(nil)
 

snow/consensus/snowball/binary_slush.go

@@ -3,9 +3,7 @@
 
 package snowball
 
-import (
-	"fmt"
-)
+import "fmt"
 
 var _ BinarySlush = (*binarySlush)(nil)