From 2946db9b49bd039050eb062b14b538d3cf7118a0 Mon Sep 17 00:00:00 2001 From: unkinben Date: Sun, 5 Jul 2026 17:11:55 +1000 Subject: [PATCH] fix pre-commit go-vet for multi-package layout The dnephin go-vet hook runs 'go vet' at the repo root, which has no Go files (all live under cmd/, internal/, pkg/), failing with 'no Go files'. Replace it with a local 'go vet ./...' hook, mirroring artifactapi. --- .pre-commit-config.yaml | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 5b65ffe..f153dcb 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -6,10 +6,22 @@ repos: - id: end-of-file-fixer - id: check-yaml - id: check-added-large-files + - id: check-merge-conflict - repo: https://github.com/dnephin/pre-commit-golang rev: v0.5.1 hooks: - id: go-fmt - - id: go-vet - id: go-mod-tidy + + # encapi has no root-level Go files (all under cmd/, internal/, pkg/), so the + # dnephin go-vet hook (which runs `go vet` at the repo root) fails with + # "no Go files". Vet the whole module instead, mirroring artifactapi. + - repo: local + hooks: + - id: go-vet + name: go vet + entry: go vet ./... + language: system + types: [go] + pass_filenames: false