Add linter

Author: jessegeens

Diff for: .github/workflows/lint.yml

@@ -0,0 +1,38 @@
+# Write me a workflow that will lint the code with golangci-lint
+
+name: lint
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  golangci:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+            go-version: '1.22'
+            cache: true
+     # Cloning outside of workspace is currently not possible, see https://github.com/actions/checkout/issues/197
+     # Therefore, we move later 
+     # (we need ../reva available due to a replace in the go.mod file)
+      - name: Cloning reva
+        uses: actions/checkout@v4
+        with:
+          repository: cs3org/reva
+          path: reva
+      - run: mv reva ../reva
+      - name: Go mod tidy
+        run: go mod tidy
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+            version: latest
+            args: --timeout=5m

Diff for: .golangci.yaml

@@ -0,0 +1,3 @@
+linters:
+  disable:
+    - errcheck

Diff for: cback/http/cback.go

@@ -48,8 +48,6 @@ func init() {
 	reva.RegisterPlugin(svc{})
 }
 
-const webdavPrefix = "/remote.php/dav/files/"
-
 type config struct {
 	Prefix            string `mapstructure:"prefix"`
 	Token             string `mapstructure:"token"`

Diff for: share/sql/sql.go

@@ -233,7 +233,7 @@ func (m *mgr) GetShare(ctx context.Context, ref *collaboration.ShareReference) (
 			return nil, err
 		}
 	default:
-		err = errtypes.NotFound(ref.String())
+		return nil, errtypes.NotFound(ref.String())
 	}
 
 	// resolve grantee's user type if applicable