为 golang repo 运行预提交挂钩时出错 [命名文件必须是 .go 文件：./...]

Question

这是我的.pre-commit-config.yam l 文件的内容，repos:- repo: local  hooks:    - id: static-checks-pramod      name: Static Analysis      description: This hook does static analysis      entry: staticcheck -tests=false ./...      language: golang      types: [text]在本地为所有本地文件运行挂钩时，我遇到了以下错误，pramodchoudhari@Pramods-MacBook-Pro-2 my-repo % pre-commit run --all-filesStatic Analysis..........................................................Failed- hook id: static-checks-pramod- exit code: 1-: named files must be .go files: ./...-: named files must be .go files: ./...-: named files must be .go files: ./...-: named files must be .go files: ./...-: named files must be .go files: ./...-: named files must be .go files: ./...-: named files must be .go files: ./...-: named files must be .go files: ./...但是如果我在本地运行 staticcheck 命令，它工作正常，如下所示，pramodchoudhari@Pramods-MacBook-Pro-2 my-repo % staticcheck -tests=false ./...         pramodchoudhari@Pramods-MacBook-Pro-2 my-repo % 我不确定我在预提交配置中做错了什么。PS：我正在使用这个linter 对我的 repo 进行静态分析

Answer 1

在您.pre-commit-config.yaml的types设置中text，它会将所有文本（如文件）传递给staticcheck，但它只期望go-files。

你可能想要types: [go]。

Answer 2

您的配置很接近，但有一些可以改进的地方。现在你正在安装一个 noopgolang存储库，然后同时运行./...（golang 中的所有内容）和text存储库中的所有文件（可能不是你想要的！）

首先让我们解决 noop 存储库——language: golang指示pre-commit它应该如何安装挂钩本身——在这种情况下你没有告诉它安装任何东西（repo: local挂钩通常additional_dependencies用来安装东西）

假设您想要预提交来管理安装（毕竟这是预提交要点的一部分——它管理您的安装，所以您不需要指导您的贡献者如何安装所有东西）——对于你会告诉 pre-commit 安装这样的东西：

    # ...

    language: golang

    additional_dependencies: [honnef.co/go/tools/cmd/staticcheck@2022.1.2]

    # ...

现在让我们来处理正在传递的文件——上面的@jkittner 正中要害，但我会详细说明一下。

pre-commit的参数模式：

您的挂钩应该期望收到该args值，然后是暂存文件列表。

然后从过滤文件类型：

text- 文件是否看起来像文本文件

将它们放在一起，您当前的配置就像运行一样staticcheck -tests=false ./... $(git ls-files)（假设您只有文本文件，我所知道的过滤掉二进制文件的 shell 方法并不是很好）

您可能想过滤掉 go 文件，并且您可能不想对每个文件进行双重检查——试试这个：

    # ...

    entry: staticcheck -tests=false

    types: [go]

    # ...

或者，如果你总是想运行所有的东西（我不推荐，它会一直变慢！）你可以关闭预提交的文件处理

    # ...

    entry: staticcheck -tests=false ./...

    pass_filenames: false

    always_run: true

    # ...

免责声明：我写了预提交