Merge branch 'feat-add-golangci-lint'

Author: fsgreco

.golangci.yaml

@@ -0,0 +1,69 @@
+version: "2"
+linters:
+  enable:
+    - containedctx
+    - copyloopvar
+    - dogsled
+    - dupl
+    - durationcheck
+    - errchkjson
+    - errorlint
+    - bodyclose
+    - exhaustive
+    - forcetypeassert
+    - gochecknoglobals
+    - goconst
+    - gocritic
+    - gomoddirectives
+    - gomodguard
+    - goprintffuncname
+    - gosec
+    - importas
+    - misspell
+    - nakedret
+    - nestif
+    - nilerr
+    - noctx
+    - nilnil
+    - nonamedreturns
+    - prealloc
+    - predeclared
+    - revive
+    - testpackage
+    - rowserrcheck
+    - sqlclosecheck
+    - tparallel
+    - unconvert
+    - unparam
+    - wastedassign
+    - whitespace
+    - wrapcheck
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    rules:
+      - linters:
+          - lll
+        path: _test\.go
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+  settings:
+    gomoddirectives:
+      replace-local: true
+      retract-allow-no-explanation: true
+formatters:
+  enable:
+    - gofmt
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+

README.md

@@ -116,7 +116,7 @@ Pull requests are welcome! Just ensure no bunnies are harmed during the process.
 
 ### Testing & Quality Assurance
 - [ ] **Add unit tests**  
-- [ ] **Set up golangci-lint** with `.golangci.yml` configuration
+- [x] **Set up golangci-lint** with `.golangci.yml` configuration
 
 ### CI/CD & Automation
 - [ ] **GitHub Actions workflow** Once tests are in place enable `ci.yml` 

cmd/bunnysign/main.go

@@ -14,17 +14,16 @@ type CLIConfig struct {
 }
 
 func parseFlags() CLIConfig {
-	clear := flag.Bool("c", false, "Clear the bunny after showing the message")
+	clearLastFrame := flag.Bool("c", false, "Clear the bunny after showing the message")
 	debug := flag.Bool("debug", false, "Debug CLI features")
 	flag.Parse()
 	return CLIConfig{
-		Clear: *clear,
+		Clear: *clearLastFrame,
 		Debug: *debug,
 	}
 }
 
 func main() {
-
 	config := parseFlags()
 
 	phrases := flag.Args() // os.Args[1:]

logupdate/logupdate.go

@@ -12,19 +12,20 @@ import (
 // CreatePlayer is a HOF: creates an animation player function that manages frame updates.
 // It takes the FrameGenerator function and returns another function that displays the animated content.
 // Thjis returned function handles clearing previous frames and displaying new ones with timing.
-func CreatePlayer(FrameGenerator func(content string) string) func(content string) int {
+func CreatePlayer(frameGenerator func(content string) string) func(content string) int {
 	previousFrameLines := 0
 
 	return func(content string) int {
 		if previousFrameLines > 0 {
 			ClearLines(previousFrameLines)
 		}
 		// print the frame
-		frame := FrameGenerator(content)
+		frame := frameGenerator(content)
 		fmt.Printf("%s\n", frame)
 
 		previousFrameLines = strings.Count(frame, "\n") + 1
 		// wait random to print the new frame
+		//nolint:gosec
 		time.Sleep(time.Duration(rand.Intn(300)+100) * time.Millisecond)
 
 		return previousFrameLines