Merge pull request #81 from omarkurt/makefile-acting-os

feat: Add OS-aware pre-commit installation

Author: omarkurt

Makefile

@@ -1,38 +1,65 @@
 SHELL := /usr/bin/env bash
 
-.PHONY: all
-all: install-pre-commit-mac \
-	install-pre-commit-linux \
-	install-pre-commit-hooks \
-	golangci \
-	yamllint \
-	govet \
-	goerrcheck \
-	gofull \
-	test-report \
-	test \
-	lint
+# OS detection variable (Darwin for macOS, Linux for Linux)
+UNAME_S := $(shell uname -s)
 
+.PHONY: all install-pre-commit-mac install-pre-commit-linux install-pre-commit install-pre-commit-hooks gofmt golangci yamllint govet goerrcheck gofull gosec test-report test lint
 
+all: install-pre-commit \
+    install-pre-commit-hooks \
+    golangci \
+    yamllint \
+    govet \
+    goerrcheck \
+    gofull \
+    test-report \
+    test \
+    lint
+
+# Runs only on macOS
 install-pre-commit-mac:
-	@brew install pre-commit
+	@echo "Checking pre-commit on macOS..."
+	@if ! command -v pre-commit &> /dev/null; then \
+		echo "pre-commit not found, installing via brew..."; \
+		brew install pre-commit; \
+	else \
+		echo "pre-commit is already installed."; \
+	fi
+
+# Runs only on Linux
 install-pre-commit-linux:
-	@sudo apt install pre-commit
+	@echo "Checking pre-commit on Linux..."
+	@if ! command -v pre-commit &> /dev/null; then \
+		echo "pre-commit not found, installing via apt..."; \
+		sudo apt update && sudo apt install -y pre-commit; \
+	else \
+		echo "pre-commit is already installed."; \
+	fi
+
+# Selects the correct installer based on the operating system
+install-pre-commit:
+ifeq ($(UNAME_S), Darwin)
+	@$(MAKE) install-pre-commit-mac
+else ifeq ($(UNAME_S), Linux)
+	@$(MAKE) install-pre-commit-linux
+else
+	@echo "Unsupported OS: $(UNAME_S). Please install pre-commit manually."
+	@exit 1
+endif
+
 install-pre-commit-hooks:
 	@pre-commit install --install-hooks
 	@pre-commit install --hook-type commit-msg --install-hooks
 
 gofmt:
 	@go fmt ./...
 
-
 golangci:
 	@golangci-lint run --config=.golangci.yaml ./...
 
 yamllint:
 	@pre-commit run yamllint --all-files
 
-
 govet:
 	@go vet ./...
 
@@ -47,7 +74,6 @@ gofull:
 gosec:
 	@gosec ./...
 
-.PHONY: gost-report
 test-report:
 	@echo "Running tests and generating coverage report..."
 	@go test -coverprofile=coverage.out -covermode=atomic -race -v ./... || (echo "\nTests failed. Check test output above."; exit 1)
@@ -59,13 +85,12 @@ test-report:
 	@go tool cover -func=coverage.out | grep total:
 	@echo "\nHTML report: file://$(shell pwd)/coverage.html"
 
-.PHONY: lint
 test: ## Run all tests
 	@go test -v -race -coverprofile=coverage.out -covermode=atomic ./...
 
 lint: ## Run all linters
 	@echo "Running linters..."
-	make yamllint
+	@$(MAKE) yamllint
 	@$(MAKE) golangci
 	@$(MAKE) goerrcheck
-	@$(MAKE) govet
+	@$(MAKE) govet