refactor: sort imports (#572)

Also added gci to .golangci.yaml to keep imports consistent in the
future.

.golangci.yml

@@ -1,5 +1,10 @@
 ---
 linters-settings:
+  gci:
+    sections:
+      - standard
+      - default
+      - prefix(github.com/hetznercloud)
   exhaustive:
     default-signifies-exhaustive: true
   gomodguard:
@@ -52,7 +57,7 @@ linters:
     - exhaustive
     # - gocritic
     # - golint
-    # - goimports
+    - gci
     - gomodguard
     - gosec
     # - gosimple

internal/cli/root.go

@@ -3,6 +3,8 @@ package cli
 import (
 	"time"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/certificate"
 	"github.com/hetznercloud/cli/internal/cmd/completion"
 	"github.com/hetznercloud/cli/internal/cmd/context"
@@ -24,7 +26,6 @@ import (
 	"github.com/hetznercloud/cli/internal/cmd/volume"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
-	"github.com/spf13/cobra"
 )
 
 func NewRootCommand(state *state.State, client hcapi2.Client) *cobra.Command {

internal/cmd/base/cmd.go

@@ -3,10 +3,11 @@ package base
 import (
 	"context"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
-	"github.com/spf13/cobra"
 )
 
 // Cmd allows defining commands for generic resource-based commands

internal/cmd/base/delete.go

@@ -6,12 +6,13 @@ import (
 	"reflect"
 	"strings"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 // DeleteCmd allows defining commands for deleting a resource.

internal/cmd/base/describe.go

@@ -8,13 +8,14 @@ import (
 	"reflect"
 	"strings"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/hetznercloud/cli/internal/cmd/output"
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 // DescribeCmd allows defining commands for describing a resource.

internal/cmd/base/labels.go

@@ -6,11 +6,12 @@ import (
 	"fmt"
 	"strings"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
-	"github.com/spf13/cobra"
 )
 
 // LabelCmds allows defining commands for adding labels to resources.

internal/cmd/base/list.go

@@ -3,14 +3,15 @@ package base
 import (
 	"context"
 	"fmt"
+
+	"github.com/spf13/cobra"
 	"github.com/spf13/pflag"
 
 	"github.com/hetznercloud/cli/internal/cmd/output"
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 // ListCmd allows defining commands for listing resources

internal/cmd/base/set_rdns.go

@@ -7,12 +7,13 @@ import (
 	"reflect"
 	"strings"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 // SetRdnsCmd allows defining commands for setting the RDNS of a resource.

internal/cmd/base/update.go

@@ -6,14 +6,14 @@ import (
 	"reflect"
 	"strings"
 
+	"github.com/spf13/cobra"
 	"github.com/spf13/pflag"
 
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 // UpdateCmd allows defining commands for updating a resource.

internal/cmd/certificate/certificate.go

@@ -1,9 +1,10 @@
 package certificate
 
 import (
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
-	"github.com/spf13/cobra"
 )
 
 func NewCommand(cli *state.State, client hcapi2.Client) *cobra.Command {

internal/cmd/certificate/create.go

@@ -4,11 +4,12 @@ import (
 	"fmt"
 	"io/ioutil"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/state"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 func newCreateCommand(cli *state.State) *cobra.Command {

internal/cmd/certificate/delete.go

@@ -3,11 +3,12 @@ package certificate
 import (
 	"context"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/base"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 var deleteCmd = base.DeleteCmd{

internal/cmd/certificate/describe.go

@@ -4,12 +4,12 @@ import (
 	"context"
 	"fmt"
 
-	"github.com/hetznercloud/cli/internal/cmd/base"
-	"github.com/hetznercloud/cli/internal/hcapi2"
+	"github.com/dustin/go-humanize"
 	"github.com/spf13/cobra"
 
-	"github.com/dustin/go-humanize"
+	"github.com/hetznercloud/cli/internal/cmd/base"
 	"github.com/hetznercloud/cli/internal/cmd/util"
+	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
 )
 

internal/cmd/certificate/list.go

@@ -2,10 +2,11 @@ package certificate
 
 import (
 	"context"
-	"github.com/spf13/pflag"
 	"strings"
 	"time"
 
+	"github.com/spf13/pflag"
+
 	"github.com/hetznercloud/cli/internal/cmd/base"
 	"github.com/hetznercloud/cli/internal/cmd/output"
 	"github.com/hetznercloud/cli/internal/cmd/util"

internal/cmd/certificate/update.go

@@ -3,12 +3,12 @@ package certificate
 import (
 	"context"
 
-	"github.com/hetznercloud/cli/internal/cmd/base"
-	"github.com/hetznercloud/cli/internal/hcapi2"
+	"github.com/spf13/cobra"
 	"github.com/spf13/pflag"
 
+	"github.com/hetznercloud/cli/internal/cmd/base"
+	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 var updateCmd = base.UpdateCmd{

internal/cmd/cmpl/suggestions_test.go

@@ -3,9 +3,10 @@ package cmpl_test
 import (
 	"testing"
 
-	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/spf13/cobra"
 	"github.com/stretchr/testify/assert"
+
+	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 )
 
 func TestSuggestCandidates(t *testing.T) {

internal/cmd/completion/completion.go

@@ -4,8 +4,9 @@ import (
 	"fmt"
 	"os"
 
-	"github.com/hetznercloud/cli/internal/state"
 	"github.com/spf13/cobra"
+
+	"github.com/hetznercloud/cli/internal/state"
 )
 
 // const (

internal/cmd/context/active.go

@@ -3,8 +3,9 @@ package context
 import (
 	"fmt"
 
-	"github.com/hetznercloud/cli/internal/state"
 	"github.com/spf13/cobra"
+
+	"github.com/hetznercloud/cli/internal/state"
 )
 
 func newActiveCommand(cli *state.State) *cobra.Command {

internal/cmd/context/context.go

@@ -1,8 +1,9 @@
 package context
 
 import (
-	"github.com/hetznercloud/cli/internal/state"
 	"github.com/spf13/cobra"
+
+	"github.com/hetznercloud/cli/internal/state"
 )
 
 func NewCommand(cli *state.State) *cobra.Command {

internal/cmd/context/create.go

@@ -7,9 +7,10 @@ import (
 	"strings"
 	"syscall"
 
-	"github.com/hetznercloud/cli/internal/state"
 	"github.com/spf13/cobra"
 	"golang.org/x/crypto/ssh/terminal"
+
+	"github.com/hetznercloud/cli/internal/state"
 )
 
 func newCreateCommand(cli *state.State) *cobra.Command {

internal/cmd/context/delete.go

@@ -3,9 +3,10 @@ package context
 import (
 	"fmt"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/hetznercloud/cli/internal/state"
-	"github.com/spf13/cobra"
 )
 
 func newDeleteCommand(cli *state.State) *cobra.Command {

internal/cmd/context/list.go

@@ -1,10 +1,11 @@
 package context
 
 import (
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/output"
 	"github.com/hetznercloud/cli/internal/cmd/util"
 	"github.com/hetznercloud/cli/internal/state"
-	"github.com/spf13/cobra"
 )
 
 var listTableOutput *output.Table

internal/cmd/context/use.go

@@ -3,9 +3,10 @@ package context
 import (
 	"fmt"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
 	"github.com/hetznercloud/cli/internal/state"
-	"github.com/spf13/cobra"
 )
 
 func newUseCommand(cli *state.State) *cobra.Command {

internal/cmd/datacenter/datacenter.go

@@ -1,9 +1,10 @@
 package datacenter
 
 import (
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
-	"github.com/spf13/cobra"
 )
 
 func NewCommand(cli *state.State, client hcapi2.Client) *cobra.Command {

internal/cmd/datacenter/describe.go

@@ -4,10 +4,11 @@ import (
 	"context"
 	"fmt"
 
+	"github.com/spf13/cobra"
+
 	"github.com/hetznercloud/cli/internal/cmd/base"
 	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/hcloud-go/v2/hcloud"
-	"github.com/spf13/cobra"
 )
 
 var describeCmd = base.DescribeCmd{

internal/cmd/datacenter/list.go

@@ -2,6 +2,7 @@ package datacenter
 
 import (
 	"context"
+
 	"github.com/spf13/pflag"
 
 	"github.com/hetznercloud/cli/internal/cmd/base"

internal/cmd/firewall/add_rule.go

@@ -5,13 +5,13 @@ import (
 	"fmt"
 	"net"
 
-	"github.com/hetznercloud/cli/internal/cmd/base"
-	"github.com/hetznercloud/cli/internal/hcapi2"
-	"github.com/hetznercloud/hcloud-go/v2/hcloud"
 	"github.com/spf13/cobra"
 
+	"github.com/hetznercloud/cli/internal/cmd/base"
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
+	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
+	"github.com/hetznercloud/hcloud-go/v2/hcloud"
 )
 
 var AddRuleCommand = base.Cmd{

internal/cmd/firewall/apply_to_resource.go

@@ -4,13 +4,13 @@ import (
 	"context"
 	"fmt"
 
-	"github.com/hetznercloud/cli/internal/cmd/base"
-	"github.com/hetznercloud/cli/internal/hcapi2"
-	"github.com/hetznercloud/hcloud-go/v2/hcloud"
 	"github.com/spf13/cobra"
 
+	"github.com/hetznercloud/cli/internal/cmd/base"
 	"github.com/hetznercloud/cli/internal/cmd/cmpl"
+	"github.com/hetznercloud/cli/internal/hcapi2"
 	"github.com/hetznercloud/cli/internal/state"
+	"github.com/hetznercloud/hcloud-go/v2/hcloud"
 )
 
 var ApplyToResourceCommand = base.Cmd{