Fix lint (#476)

Didn't run for some reasons; something with the actions cache:

	/usr/bin/tar --use-compress-program zstd -d -xf /home/runner/work/_temp/b9a1abd7-273e-449f-b255-5e8e947b3f97/cache.tzst -P -C /home/runner/work/fsnotify/fsnotify
	Error: /usr/bin/tar: ../../../go/pkg/mod/golang.org/x/sys@v0.0.0-20220412211240-33da011f77ad/windows/race0.go: Cannot open: File exists
	Error: /usr/bin/tar: ../../../go/pkg/mod/golang.org/x/sys@v0.0.0-20220412211240-33da011f77ad/windows/types_windows.go: Cannot open: File exists

Replace it with staticcheck; which is nicer anyway.

.github/workflows/build.yml

@@ -1,15 +1,11 @@
-name: build
-on:
-  push:
-  pull_request:
+name: 'build'
+on:   ['push', 'pull_request']
 jobs:
-  cross-compile-supported:
+  cross-compile:
     strategy:
       fail-fast: false
       matrix:
-        go:
-          - '1.16'
-          - '1.18'
+        go: ['1.16', '1.18']
     runs-on: ubuntu-latest
     steps:
       - name: setup Go

.github/workflows/staticcheck.yml

@@ -0,0 +1,13 @@
+name: 'staticcheck'
+on:   ['push', 'pull_request']
+jobs:
+  staticcheck:
+    name:    'staticcheck'
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+      with:
+        fetch-depth: 1
+    - uses: dominikh/staticcheck-action@v1.2.0
+      with:
+        version: '2022.1'

.github/workflows/test.yml

@@ -1,7 +1,5 @@
-name: test
-on:
-  push:
-  pull_request:
+name: 'test'
+on:   ['push', 'pull_request']
 jobs:
   test:
     strategy:

.github/workflows/vagrant.yml

@@ -1,7 +1,5 @@
-name: vagrant
-on:
-  push:
-  pull_request:
+name: 'test'
+on:   ['push', 'pull_request']
 jobs:
   test:
     strategy:

inotify_test.go

@@ -132,7 +132,7 @@ func TestInotifyCloseCreate(t *testing.T) {
 	}
 	h.Close()
 	select {
-	case _ = <-w.Events:
+	case <-w.Events:
 	case err := <-w.Errors:
 		t.Fatalf("Error from watcher: %v", err)
 	case <-time.After(50 * time.Millisecond):
@@ -144,7 +144,7 @@ func TestInotifyCloseCreate(t *testing.T) {
 	// Now we try to swap the file descriptor under its nose.
 	w.Close()
 	w, err = NewWatcher()
-	defer w.Close()
+	defer func() { _ = w.Close() }()
 	if err != nil {
 		t.Fatalf("Failed to create second watcher: %v", err)
 	}
@@ -351,7 +351,7 @@ func TestInotifyInnerMapLength(t *testing.T) {
 	if err != nil {
 		t.Fatalf("Failed to remove testFile: %v", err)
 	}
-	_ = <-w.Events                      // consume Remove event
+	<-w.Events                          // consume Remove event
 	<-time.After(50 * time.Millisecond) // wait IN_IGNORE propagated
 
 	func() {