add the revive linter and fix findings

.github/workflows/sanity.yml

@@ -27,15 +27,14 @@ jobs:
     - name: Setup Go
       uses: actions/setup-go@v4
       with:
-        go-version: 1.20
+        go-version: "1.20"
 
     - name: lint
       uses: golangci/golangci-lint-action@v3
-
-    - name: ginkgo-lint
-      run: |-
-        go install github.com/nunnatsa/ginkgolinter/cmd/ginkgolinter@latest
-        ginkgolinter ./...
+      with:
+        version: v1.53.2
+        skip-pkg-cache: true
+        args: --timeout=5m --out-${NO_FUTURE}format line-number
 
     - name: test
       run: make test

.golangci.yml

@@ -1,3 +1,4 @@
 linters:
   enable:
   - ginkgolinter
+  - revive

controller/controller.go

@@ -57,7 +57,7 @@ func (c *Controller) do() {
 	c.screenEvents <- msg
 
 	for {
-		var change *state.Change = nil
+		var change *state.Change
 
 		select {
 		case <-signals:

hat/hat.go

@@ -217,7 +217,6 @@ func findJoystickDeviceFile() (string, error) {
 
 	if foundDevice && len(deviceName) > 0 {
 		return eventFilePrefix + deviceName, nil // found
-	} else {
-		return "", fmt.Errorf("can't find the joystick device")
 	}
+	return "", fmt.Errorf("can't find the joystick device")
 }

main.go

@@ -80,7 +80,6 @@ func main() {
 
 	if err := server.ListenAndServe(); err != http.ErrServerClosed {
 		log.Panic(err)
-	} else {
-		fmt.Println("\nGood Bye!")
 	}
+	fmt.Println("\nGood Bye!")
 }

notifier/notifier.go

@@ -10,7 +10,7 @@ type idProvider struct {
 	counter uint64
 }
 
-func newIdProvider() *idProvider {
+func newIDProvider() *idProvider {
 	return &idProvider{
 		counter: 0,
 	}
@@ -29,7 +29,7 @@ type Notifier struct {
 func NewNotifier() *Notifier {
 	return &Notifier{
 		clientMap: make(map[uint64]chan []byte),
-		idp:       newIdProvider(),
+		idp:       newIDProvider(),
 		lock:      &sync.Mutex{},
 	}
 }