fix(codestyle): update to golangci 39

Author: jguerreiro

.golangci.yml

@@ -18,11 +18,9 @@ linters:
     - gosimple
     - govet
     - ineffassign
-    - interfacer
     - lll
     - misspell
     - nakedret
-    - scopelint
     - staticcheck
     - structcheck
     - stylecheck
@@ -39,6 +37,7 @@ linters:
     - exhaustivestruct
     - errorlint
     # temporary disables
+    - gci
     - gocritic
     - gofumpt
     - funlen
@@ -59,8 +58,6 @@ linters-settings:
     min-confidence: 0
   gocyclo:
     min-complexity: 15
-  maligned:
-    suggest-new: true
   dupl:
     threshold: 100
   goconst:
@@ -75,14 +72,3 @@ linters-settings:
       - opinionated
       - performance
       - style
-  funlen:
-    lines: 100
-    statements: 50
-
-issues:
-  exclude:
-    - "G402:" # Insecure TLS Config
-    - "G404:"
-    - "`Mapping` is a global variable"
-    - "mnd: Magic number: 1, in <argument> detected"
-    - "mnd: Magic number: 2, in <argument> detected"

cmd/dna-collector/main.go

@@ -22,9 +22,11 @@ type config struct {
 	GitlabToken string `env:"GITLAB_TOKEN"`
 }
 
+const MaxPipelineEvents = 100
+
 func runExtract(pipeline *dnacollector.Pipeline, user string) chan dnacollector.PipelineEvent {
 	// buffer it a bit so it won't block if this is going too fast
-	ch := make(chan dnacollector.PipelineEvent, 100)
+	ch := make(chan dnacollector.PipelineEvent, MaxPipelineEvents)
 
 	go func(eventChannel chan dnacollector.PipelineEvent) {
 		defer close(eventChannel)