chore: fix issues with golangci-lint (#613) (#614)

* chore: go vet is already called by golangci-lint

See dnephin/pre-commit-golang#62 (comment)
and https://golangci-lint.run/usage/linters/

* chore: bump precommit-golang

* chore: run golangci-lint for modified files

Enable the fast linter preset and only run it on newly-modified files,
and fix problems as they arise, if the linter supports it

* chore: fix line endings

* chore: expllicitly set filepaths for gherkin files

* chore: remove trail comma in arrays' last items

* chore: add empty Go file to the root directory

Golangci-lint is able to continue

* chore: add Go license to empty file

Author: mdelapenya

.pre-commit-config.yaml

@@ -17,13 +17,13 @@ repos:
             )$
 
 -   repo: git://github.com/dnephin/pre-commit-golang
-    rev: v0.3.4
+    rev: v0.3.5
     hooks:
     -   id: go-fmt
     -   id: go-lint
-    -   id: go-vet
     -   id: no-go-testing
     -   id: golangci-lint
+        args: ["--fix", "--fast", "--new"]
 
 -   repo: [email protected]:elastic/apm-pipeline-library
     rev: current
@@ -44,4 +44,7 @@ repos:
     -   id: remove-en-dashes
     -   id: check-jjbb
     -   id: check-gherkin-lint
-        args: ["--disable", "AvoidOutlineForSingleExample,TooClumsy,TooLongStep,TooManyDifferentTags,TooManySteps"]
+        args: [
+            "--disable", "AvoidOutlineForSingleExample,TooClumsy,TooLongStep,TooManyDifferentTags,TooManySteps",
+            "e2e/_suites/**/*.feature",
+        ]

.vscode/launch.json

@@ -9,7 +9,7 @@
             "program": "${workspaceFolder}/cli",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["run", "-h"]
         },
@@ -21,7 +21,7 @@
             "program": "${workspaceFolder}/cli",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["sync", "integrations", "--remote", "elastic:master"]
         },
@@ -33,7 +33,7 @@
             "program": "${workspaceFolder}/cli",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["run", "service", "mysql", "--version", "5.6"]
         },
@@ -45,7 +45,7 @@
             "program": "${workspaceFolder}/cli",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["stop", "service", "mysql", "--version", "5.6"]
         },
@@ -57,7 +57,7 @@
             "program": "${workspaceFolder}/cli",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["deploy", "redis", "--version", "4.0.11", "--profile", "metricbeat"]
         },
@@ -69,7 +69,7 @@
             "program": "${workspaceFolder}/cli",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["undeploy", "redis", "--profile", "metricbeat"]
         },
@@ -81,7 +81,7 @@
             "program": "${workspaceFolder}/cli",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["run", "profile", "metricbeat", "-v", "7.5.0", "--withServices", "apache:2.2,redis:3.2.12"]
         },
@@ -93,7 +93,7 @@
             "program": "${workspaceFolder}/cli",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["stop", "profile", "metricbeat"]
         },
@@ -105,7 +105,7 @@
             "program": "${file}",
             "env": {
                 "GO111MODULE": "on",
-                "OP_LOG_LEVEL": "DEBUG",
+                "OP_LOG_LEVEL": "DEBUG"
             },
             "args": ["--godog.format", "pretty", "metricbeat"]
         }

cli/Building.md

@@ -7,4 +7,4 @@ The `build-cli.sh` script builds the binary based on a few environment variables
 $ GOOS=("darwin" "linux" "windows") ./.ci/scripts/build-cli.sh
 # and/or
 $ GOARCH=("386" "amd64") ./.ci/scripts/build-cli.sh
-```
+```

cli/LICENSE.txt

@@ -1,3 +1,3 @@
 Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
 or more contributor license agreements. Licensed under the Elastic License;
-you may not use this file except in compliance with the Elastic License.
+you may not use this file except in compliance with the Elastic License.

doc.go

@@ -0,0 +1,5 @@
+// Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+// or more contributor license agreements. Licensed under the Elastic License;
+// you may not use this file except in compliance with the Elastic License.
+
+package main

e2e/LICENSE.txt

@@ -1,3 +1,3 @@
 Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
 or more contributor license agreements. Licensed under the Elastic License;
-you may not use this file except in compliance with the Elastic License.
+you may not use this file except in compliance with the Elastic License.

e2e/_suites/metricbeat/configurations/vsphere.yml

@@ -9,4 +9,4 @@ metricbeat.modules:
     hosts: ["https://vsphere:443/sdk"]
     username: "user"
     password: "pass"
-    insecure: true
+    insecure: true

go.mod

@@ -0,0 +1,3 @@
+module github.com/elastic/e2e-testing
+
+go 1.14

notice/NOTICE.txt.tmpl

@@ -27,4 +27,4 @@ Third party libraries used by the Elastic Beats project:
 Indirect dependencies
 
 {{ template "depInfo" .Indirect }}
-{{ end }}
+{{ end }}

notice/rules.json

@@ -15,4 +15,4 @@
         "EPL-1.0",
         "GPL-3.0"
     ]
-}
+}