From bd7f3703792ed7cd4dc0758838047728fdbef53e Mon Sep 17 00:00:00 2001
From: Hector <dev@hsmith.org>
Date: Sun, 17 Oct 2021 18:14:57 +0100
Subject: [PATCH 1/4] update command names in makefile

Rename several commands in the project makefile to follow a more consistent
naming scheme.
Update all references to changed commands.
---
 .gitlab-ci.yml  | 10 +++++-----
 .goreleaser.yml |  2 +-
 Makefile        | 26 ++++++++++++++------------
 3 files changed, 20 insertions(+), 18 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index ec49e60..b9edbfd 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1,7 +1,7 @@
 image: golang:latest
 
 before_script:
-  - make install-deps
+  - make go/dependencies
 
 stages:
   - test
@@ -10,12 +10,12 @@ stages:
 format:
   stage: test
   script:
-    - make format
+    - make go/checkFmt
 
 test:
   stage: test
   script:
-    - make test
+    - make go/test
 
 build:
   stage: build
@@ -44,6 +44,6 @@ docker-gitlab:
     - apk add make
     - docker login -u $CI_REGISTRY_USER -p $CI_REGISTRY_PASSWORD $CI_REGISTRY
   script:
-    - make docker/build-latest
-    - make docker/build-tag
+    - make docker/buildLatest
+    - make docker/buildTag
     - docker push registry.gitlab.com/hectorjsmith/fail2ban-prometheus-exporter
diff --git a/.goreleaser.yml b/.goreleaser.yml
index 366dc27..0113e8d 100644
--- a/.goreleaser.yml
+++ b/.goreleaser.yml
@@ -2,7 +2,7 @@
 # Make sure to check the documentation at http://goreleaser.com
 before:
   hooks:
-    - make install-deps
+    - make go/dependencies
 builds:
   -
     dir: src
diff --git a/Makefile b/Makefile
index 097b4cd..ed1f65d 100644
--- a/Makefile
+++ b/Makefile
@@ -1,20 +1,22 @@
-install-deps:
+go/dependencies:
 	cd src/ && go mod download
 
-# Standard go test
-test:
-	cd src/ && go test ./... -v -race
-
 # Make sure no unnecessary dependencies are present
-go-mod-tidy:
+go/checkDependencies:
 	cd src/ && go mod tidy -v
 	git diff-index --quiet HEAD
 
-format:
-	cd src/ && go fmt $(go list ./... | grep -v /vendor/)
-	cd src/ && go vet $(go list ./... | grep -v /vendor/)
+# Standard go test
+go/test:
+	cd src/ && go test ./... -v -race
 
-generateChangelog:
+go/fmt:
+	cd src/ && go fmt ./...
+
+go/checkFmt:
+	test -z $(shell gofmt -l .)
+
+docs/genChangelog:
 	./tools/git-chglog_linux_amd64 --config tools/chglog/config.yml 0.0.0.. > CHANGELOG_gen.md
 
 build/snapshot:
@@ -27,8 +29,8 @@ build/docker:
 	cd src/ && go build -o exporter \
      -ldflags '-X main.version=$(shell git describe --tags) -X main.commit=${shell git rev-parse HEAD} -X "main.date=${shell date --rfc-3339=seconds}" -X main.builtBy=docker' exporter.go
 
-docker/build-latest:
+docker/buildLatest:
 	docker build -t registry.gitlab.com/hectorjsmith/fail2ban-prometheus-exporter:latest .
 
-docker/build-tag:
+docker/buildTag:
 	docker build -t registry.gitlab.com/hectorjsmith/fail2ban-prometheus-exporter:$(shell git describe --tags) .

From 04b986811dcb8ecf32fda051096970ced80c7cc8 Mon Sep 17 00:00:00 2001
From: Hector <dev@hsmith.org>
Date: Sun, 17 Oct 2021 18:22:07 +0100
Subject: [PATCH 2/4] rename make commands for docker jobs

---
 .gitlab-ci.yml | 4 ++--
 Makefile       | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index b9edbfd..236b8e5 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -44,6 +44,6 @@ docker-gitlab:
     - apk add make
     - docker login -u $CI_REGISTRY_USER -p $CI_REGISTRY_PASSWORD $CI_REGISTRY
   script:
-    - make docker/buildLatest
-    - make docker/buildTag
+    - make docker/build/latest
+    - make docker/build/tag
     - docker push registry.gitlab.com/hectorjsmith/fail2ban-prometheus-exporter
diff --git a/Makefile b/Makefile
index ed1f65d..f90f80e 100644
--- a/Makefile
+++ b/Makefile
@@ -29,8 +29,8 @@ build/docker:
 	cd src/ && go build -o exporter \
      -ldflags '-X main.version=$(shell git describe --tags) -X main.commit=${shell git rev-parse HEAD} -X "main.date=${shell date --rfc-3339=seconds}" -X main.builtBy=docker' exporter.go
 
-docker/buildLatest:
+docker/build/latest:
 	docker build -t registry.gitlab.com/hectorjsmith/fail2ban-prometheus-exporter:latest .
 
-docker/buildTag:
+docker/build/tag:
 	docker build -t registry.gitlab.com/hectorjsmith/fail2ban-prometheus-exporter:$(shell git describe --tags) .

From bb9531f6b5a92a20b61466b5fdee4666056fa916 Mon Sep 17 00:00:00 2001
From: Hector <dev@hsmith.org>
Date: Sun, 17 Oct 2021 18:25:03 +0100
Subject: [PATCH 3/4] add gitlab ci job to check dependencies

---
 .gitlab-ci.yml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 236b8e5..559c0fe 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -7,6 +7,11 @@ stages:
   - test
   - build
 
+dependencies:
+  stage: test
+  script:
+    - make go/checkDependencies
+
 format:
   stage: test
   script:

From cf856a23dcfce036258fc88720ab90b9ae97450c Mon Sep 17 00:00:00 2001
From: Hector <dev@hsmith.org>
Date: Sun, 17 Oct 2021 18:30:56 +0100
Subject: [PATCH 4/4] rename docker gitlab ci step

---
 .gitlab-ci.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 559c0fe..0be82e2 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -36,7 +36,7 @@ build:
       - dist/checksums.txt
     expire_in: 1 day
 
-docker-gitlab:
+docker/gitlab:
   stage: build
   only:
     - main