2

我正在尝试将 flakehell 作为预提交挂钩运行。

我的 .pre-commit-config.yaml：

repos:
-   repo: local
    hooks:             
    -   id: flakehell
        name: flakehell
        entry: flakehell lint project/
        language: python
        pass_filenames: true
        stages: [push, commit]
        verbose: true

pyproject.toml：

[tool.flakehell]
exclude = ["README.md"]
format = "colored"
show_source = true

[tool.flakehell.plugins]
flake8-bandit = ["+*", "-S322"]
flake8-bugbear = ["+*"]
flake8-builtins = ["+*"]
flake8-comprehensions = ["+*"]
flake8-darglint = ["+*"]
flake8-docstrings = ["+*"]
flake8-eradicate = ["+*"]
flake8-isort = ["+*"]
flake8-mutable = ["+*"]
flake8-pytest-style = ["+*"]
flake8-spellcheck = ["+*"]
mccabe = ["+*"]
pep8-naming = ["+*"]
pycodestyle = ["+*"]
pyflakes = ["+*"]
pylint = ["+*"]

还是有一些问题。当我尝试使用 git commit 时，我看到 flakehell 开始工作。几秒钟后它失败了，我得到了错误：

“UnicodeDecodeError：‘utf-8’编解码器无法解码位置 10 中的字节 0xb8：无效的起始字节”

没有“预提交”它会很好用（我的意思是当我输入“flakehell lint”时）

甚至可以将 flakehell 配置为预提交吗？

如果没有 - 您是否为 python 项目推荐任何更好/有效的解决方案？

4

1 回答 1

6

您的配置不正确，您没有限制传递给您的钩子的文件，files因此types它默认为您存储库中的所有文件。大概你有一些二进制文件正在传递给 flakehell

我还注意到您的配置既传递了路径又传递了pass_filenames: true（pass_filenames: true是默认值，因此您不应该使用它）

你要么想在 args 中列出路径（不推荐，因为你总是 lint 比你正在改变的更多）或者你想正确过滤文件名

此外，verbose: true它不适合在调试之外使用，因为它会在输出中增加警告噪音

此外，您不是通过预提交管理 flakehell 的安装，这会给您的贡献者增加额外的负担，以尝试在本地设置任何开发环境，预提交的大部分要点是它管理安装您的工具，因此您的贡献者不必费力地进行正确的格式化/ linting 设置（消除一整类“它在我的机器上工作”的问题）

此外，看起来 flakehell 直接支持预提交，所以你不需要repo: local像你一样使用逃生舱口

把所有这些放在一起，你可能想要这样的东西：

repos:
-   repo: https://github.com/life4/flakehell
    rev: v.0.7.1
    hooks:
    -   id: flakehell

如果您想使用本地配置，您可能需要以下内容：

repos:
-   repo: local
    hooks:             
    -   id: flakehell
        name: flakehell
        entry: flakehell lint
        types: [python]
        files: ^project/
        language: python
        additional_dependencies: [flakehell==0.7.1]
        stages: [push, commit]

免责声明：我是预提交的创建者

于 2021-01-11T08:36:39.333 回答