feat: add goimports linter, and format files with goimport -w

Author: lazysegtree

.golangci.yaml

@@ -300,7 +300,7 @@ linters:
     # Todo enable - too much for us
     #- godot # checks if comments end in a period
     # Todo enable - one small fix
-    #- goimports # in addition to fixing imports, goimports also formats your code in the same style as gofmt
+    - goimports # in addition to fixing imports, goimports also formats your code in the same style as gofmt
     - gomoddirectives # manages the use of 'replace', 'retract', and 'excludes' directives in go.mod
     - goprintffuncname # checks that printf-like functions are named with f at the end
     # Todo enable - ignore for a few int overflow, and fix file permissions

main.go

@@ -2,6 +2,7 @@ package main
 
 import (
 	"embed"
+
 	"github.com/yorukot/superfile/src/cmd"
 )
 

src/cmd/main.go

@@ -4,8 +4,6 @@ import (
 	"embed"
 	"encoding/json"
 	"fmt"
-	"github.com/yorukot/superfile/src/internal/common"
-	"github.com/yorukot/superfile/src/internal/common/utils"
 	"io"
 	"log"
 	"log/slog"
@@ -14,6 +12,9 @@ import (
 	"runtime"
 	"time"
 
+	"github.com/yorukot/superfile/src/internal/common"
+	"github.com/yorukot/superfile/src/internal/common/utils"
+
 	tea "github.com/charmbracelet/bubbletea"
 	"github.com/charmbracelet/lipgloss"
 	"github.com/urfave/cli/v2"

src/internal/common/predefined_variable.go

@@ -1,10 +1,11 @@
 package common
 
 import (
+	"time"
+
 	"github.com/charmbracelet/lipgloss"
 	"github.com/charmbracelet/x/exp/term/ansi"
 	"github.com/yorukot/superfile/src/config/icon"
-	"time"
 )
 
 const WheelRunTime = 5

src/internal/common/utils/file_utils.go

@@ -1,9 +1,12 @@
 package utils
 
-import "os"
-import "fmt"
-import "github.com/pelletier/go-toml/v2"
-import "reflect"
+import (
+	"fmt"
+	"os"
+	"reflect"
+
+	"github.com/pelletier/go-toml/v2"
+)
 
 // Utility functions related to file operations
 

src/internal/common/utils/log_utils.go

@@ -1,7 +1,9 @@
 package utils
 
-import "log/slog"
-import "os"
+import (
+	"log/slog"
+	"os"
+)
 
 // Todo : Eventually we want to remove all such usage that can result in app exiting abruptly
 func LogAndExit(msg string, values ...any) {

src/internal/common/utils/shell_utils.go

@@ -4,11 +4,12 @@ import (
 	"context"
 	"errors"
 	"fmt"
-	variable "github.com/yorukot/superfile/src/config"
 	"log/slog"
 	"os/exec"
 	"runtime"
 	"time"
+
+	variable "github.com/yorukot/superfile/src/config"
 )
 
 // Choose correct shell as per OS

src/internal/config_function.go

@@ -1,13 +1,14 @@
 package internal
 
 import (
-	"github.com/yorukot/superfile/src/internal/common/utils"
 	"log/slog"
 	"os"
 	"path/filepath"
 	"runtime"
 	"strings"
 
+	"github.com/yorukot/superfile/src/internal/common/utils"
+
 	"github.com/barasher/go-exiftool"
 	variable "github.com/yorukot/superfile/src/config"
 	"github.com/yorukot/superfile/src/config/icon"

src/internal/handle_panel_movement.go

@@ -1,13 +1,14 @@
 package internal
 
 import (
-	"github.com/yorukot/superfile/src/internal/common/utils"
 	"log/slog"
 	"os"
 	"os/exec"
 	"path/filepath"
 	"runtime"
 
+	"github.com/yorukot/superfile/src/internal/common/utils"
+
 	variable "github.com/yorukot/superfile/src/config"
 )
 

src/internal/handle_panel_navigation.go

@@ -4,11 +4,12 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
-	"github.com/yorukot/superfile/src/internal/common"
 	"log/slog"
 	"os"
 	"path/filepath"
 
+	"github.com/yorukot/superfile/src/internal/common"
+
 	variable "github.com/yorukot/superfile/src/config"
 )