1599 update and fix linter

.golangci.yml

@@ -50,7 +50,6 @@ linters:
   # inverted configuration with `enable-all` and `disable` is not scalable during updates of golangci-lint
   disable-all: true
   enable:
-    - deadcode
     - depguard
     - dogsled
     - gochecknoinits
@@ -64,7 +63,6 @@ linters:
     - ineffassign
     - nakedret
     - unconvert
-    - varcheck
     - whitespace
     - gocognit
     - prealloc
@@ -82,12 +80,14 @@ issues:
         - gocritic
         - ineffassign
         - govet
+    - path: main.go
+      linters:
         - typecheck
   new: false
 
 # golangci.com configuration
 # https://github.com/golangci/golangci/wiki/Configuration
 service:
-  golangci-lint-version: 1.43.x # use the fixed version to not introduce new linters unexpectedly
+  golangci-lint-version: 1.52.x # use the fixed version to not introduce new linters unexpectedly
   prepare:
-    - echo "here I can run custom commands, but no preparation needed for this repo"
+    - echo "here I can run custom commands, but no preparation needed for this repo"

Makefile

@@ -18,8 +18,9 @@ audit:
 build: generate-prod
 	go build -tags 'production' $(LDFLAGS) -o $(BINPATH)/dp-frontend-feedback-controller
 
+.PHONY: lint 
 lint:
-	go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.45.2
+	go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.52.2
 	golangci-lint run ./...
 
 .PHONY: debug

handlers/feedback.go

@@ -28,7 +28,7 @@ func (f *Feedback) FeedbackThanks() http.HandlerFunc {
 	})
 }
 
-func feedbackThanks(w http.ResponseWriter, req *http.Request, url string, rend interfaces.Renderer, cacheHelperService *cacheHelper.Helper, lang string) {
+func feedbackThanks(w http.ResponseWriter, req *http.Request, uri string, rend interfaces.Renderer, cacheHelperService *cacheHelper.Helper, lang string) {
 	ctx := req.Context()
 	var wholeSite string
 
@@ -40,7 +40,7 @@ func feedbackThanks(w http.ResponseWriter, req *http.Request, url string, rend i
 	}
 
 	basePage := rend.NewBasePageModel()
-	p := mapper.CreateGetFeedbackThanks(req, basePage, lang, url, wholeSite)
+	p := mapper.CreateGetFeedbackThanks(req, basePage, lang, uri, wholeSite)
 
 	if cfg.EnableNewNavBar {
 		mappedNavContent, err := cacheHelperService.GetMappedNavigationContent(ctx, lang)

main.go

@@ -69,7 +69,7 @@ func main() {
 	cacheService, err := cacheHelper.Init(ctx, cacheConfig)
 	cacheService.RunUpdates(ctx, svcErrors)
 
-	// nolint: typecheck
+	//nolint:typecheck
 	rend := render.NewWithDefaultClient(assets.Asset, assets.AssetNames, cfg.PatternLibraryAssetsPath, cfg.SiteDomain)
 
 	routes.Setup(ctx, r, cfg, rend, healthcheck, cacheService)

mocks/mocks.go

@@ -17,6 +17,6 @@ var enLocale = []string{
 }
 
 // MockAssetFunction returns mocked toml []bytes
-func MockAssetFunction(name string) ([]byte, error) {
+func MockAssetFunction(name string) ([]byte, error) { //nolint:all // app does not use welsh
 	return []byte(strings.Join(enLocale, "\n")), nil
 }