setup linting configuration

Author: teaglebuilt

.github/FUNDING.yml

@@ -1,3 +1,4 @@
 # These are supported funding model platforms
 
 github: teaglebuilt
+buy_me_a_coffee: teaglebuilt

.github/workflows/release.yaml

@@ -0,0 +1,67 @@
+---
+name: Release Workflow
+
+on:
+  push:
+    tags:
+    - "v*.*.*"
+
+permissions:
+  contents: write
+
+jobs:
+  release:
+    name: Build & Release GH CLI Extension
+    runs-on: ubuntu-latest
+    steps:
+    - name: Checkout Code
+      uses: actions/checkout@v4
+      with:
+        fetch-depth: 0
+
+    - name: Setup Go
+      uses: actions/setup-go@v5
+      with:
+        go-version: stable
+
+    - name: Generate Changelog
+      id: changelog
+      uses: mikepenz/[email protected]
+      with:
+        configuration: |
+          categories:
+            - title: "🚀 Features"
+              labels:
+                - feature
+            - title: "🐛 Fixes"
+              labels:
+                - fix
+            - title: "📖 Documentation"
+              labels:
+                - docs
+            - title: "🧰 Maintenance"
+              labels:
+                - chore
+                - ci
+                - refactor
+          template: |
+            ## What's Changed
+            $CHANGES
+          pr_template: "- #$NUMBER $TITLE @$AUTHOR"
+          outputFile: CHANGELOG.md
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
+    - name: Precompile GH Extension
+      uses: cli/gh-extension-precompile@v2
+      with:
+        go-version: stable
+        binary-name: 'gh-subrepo'
+
+  - name: Create GitHub Release
+    uses: softprops/action-gh-release@v2
+    with:
+      name: Release ${{ github.ref_name }}
+      tag_name: ${{ github.ref }}
+      body_path: CHANGELOG.md
+      files: dist/*

.golangci.yaml

@@ -1,25 +1,23 @@
 ---
+version: "2"
+
 linters:
   enable:
   - bodyclose
-  - depguard
-  - errorlint
   - copyloopvar
   - importas
-  - gci
-  - gofumpt
   - gocritic
-  - gosec
   - govet
   - misspell
-  - revive
-  - stylecheck
-  - testifylint
   - unconvert
+  # - errorlint
+  # - depguard
+  # - gosec
+  # - testifylint
+  # - revive
 
 formatters:
   enable:
-  - gci
   - gofmt
   - gofumpt
   - goimports

.vscode/settings.json

@@ -0,0 +1,9 @@
+{
+  "gopls": {
+    "ui.semanticTokens": true,
+    "formatting.gofumpt": true 
+  },
+  "[go]": {
+    "editor.formatOnSave": true
+  }
+}

cmd/clone/clone.go

@@ -22,7 +22,8 @@ func CloneCmd() *cobra.Command {
 
 func cloneRepository(cmd *cobra.Command, args []string) {
 	repoURL := args[0]
-	subdir := "."
+	var subdir string
+
 	if len(args) == 2 {
 		subdir = args[1]
 	} else {

cmd/pull/pull.go

@@ -41,7 +41,12 @@ func PullCmd() *cobra.Command {
 				fmt.Printf("Failed to create temp dir: %v\n", err)
 				os.Exit(1)
 			}
-			defer os.RemoveAll(tmpDir)
+
+			defer func() {
+				if err := os.RemoveAll(tmpDir); err != nil {
+					fmt.Printf("Warning: failed to remove tmp dir: %v\n", err)
+				}
+			}()
 
 			fmt.Printf("Pulling latest changes from %s...\n", remote)
 			if err := utils.ExecCmd("", "git", "clone", "--depth=1", "-b", branch, remote, tmpDir); err != nil {

cmd/status/status.go

@@ -31,7 +31,7 @@ func StatusCmd() *cobra.Command {
 			}
 
 			for _, subdir := range subdirs {
-				checkStatus(repoRoot, subdir)
+				checkStatus(subdir)
 			}
 		},
 	}
@@ -40,16 +40,19 @@ func StatusCmd() *cobra.Command {
 
 func findAllSubrepos(root string) []string {
 	var dirs []string
-	filepath.WalkDir(root, func(path string, d os.DirEntry, err error) error {
+	err := filepath.WalkDir(root, func(path string, d os.DirEntry, err error) error {
 		if filepath.Base(path) == ".gitrepo" {
 			dirs = append(dirs, filepath.Dir(path))
 		}
 		return nil
 	})
+	if err != nil {
+		fmt.Printf("Error walking directory: %v\n", err)
+	}
 	return dirs
 }
 
-func checkStatus(repoRoot, subdir string) {
+func checkStatus(subdir string) {
 	gitrepoFile := filepath.Join(subdir, ".gitrepo")
 
 	cfg, err := ini.Load(gitrepoFile)
@@ -91,5 +94,5 @@ func execCommandOutput(dir, name string, args ...string) (string, error) {
 	if err != nil {
 		return "", err
 	}
-	return string(output[:40]), nil // 40-char hash
+	return string(output[:40]), nil
 }

cmd/status/status_test.go

@@ -36,7 +36,8 @@ func TestStatusCmd_Integration(t *testing.T) {
 	setupGitConfig(t, tmpWork)
 
 	readmePath := filepath.Join(tmpWork, "README.md")
-	os.WriteFile(readmePath, []byte("Initial remote content"), 0644)
+	assert.NoError(t, os.WriteFile(readmePath, []byte("Initial remote content"), 0644))
+
 	assert.NoError(t, utils.ExecCmd(tmpWork, "git", "add", "."))
 	assert.NoError(t, utils.ExecCmd(tmpWork, "git", "commit", "-m", "Initial commit"))
 	assert.NoError(t, utils.ExecCmd(tmpWork, "git", "push", "-u", "origin", "main"))
@@ -52,13 +53,13 @@ func TestStatusCmd_Integration(t *testing.T) {
 	assert.NoError(t, utils.ExecCmd(rootDir, "git", "add", "."))
 	assert.NoError(t, utils.ExecCmd(rootDir, "git", "commit", "-m", "Add subrepo"))
 
-	checkStatus(rootDir, subrepoPath)
+	checkStatus(subrepoPath)
 
 	changelogPath := filepath.Join(tmpWork, "CHANGELOG.md")
 	os.WriteFile(changelogPath, []byte("Changelog added"), 0644)
 	assert.NoError(t, utils.ExecCmd(tmpWork, "git", "add", "."))
 	assert.NoError(t, utils.ExecCmd(tmpWork, "git", "commit", "-m", "Add CHANGELOG"))
 	assert.NoError(t, utils.ExecCmd(tmpWork, "git", "push", "origin", "main"))
 
-	checkStatus(rootDir, subrepoPath)
+	checkStatus(subrepoPath)
 }