是否考虑给项目添加上pre-commit检查？

[fan-tastic-z]

目前项目我在本地添加了一下，代码中有比较多的可能是拼写问题？代码在commit 上加一些预检查会好点？方便及时发现一些测试的，format 等问题

[fan-tastic-z]

试了一下，目前的问题好像还挺多

[fan-tastic-z]

我自己常用的pre-commit-config.yaml 配置文件

fail_fast: false
repos:
  - repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v4.6.0
    hooks:
      - id: check-byte-order-marker
      - id: check-case-conflict
      - id: check-merge-conflict
      - id: check-symlinks
      - id: check-yaml
      - id: end-of-file-fixer
      - id: mixed-line-ending
      - id: trailing-whitespace
  - repo: https://github.com/commitizen-tools/commitizen
    rev: v3.28.0
    hooks:
      - id: commitizen
  - repo: https://github.com/psf/black
    rev: 24.8.0
    hooks:
      - id: black
  - repo: local
    hooks:
      - id: cargo-fmt
        name: cargo fmt
        description: Format files with rustfmt.
        entry: bash -c 'cargo fmt -- --check'
        language: rust
        files: \.rs$
        args: []
      - id: cargo-deny
        name: cargo deny check
        description: Check cargo dependencies
        entry: bash -c 'cargo deny check -d'
        language: rust
        files: \.rs$
        args: []
      - id: typos
        name: typos
        description: check typo
        entry: bash -c 'typos'
        language: rust
        files: \.*$
        pass_filenames: false
      - id: cargo-check
        name: cargo check
        description: Check the package for errors.
        entry: bash -c 'cargo check --all'
        language: rust
        files: \.rs$
        pass_filenames: false
      - id: cargo-clippy
        name: cargo clippy
        description: Lint rust sources
        entry: bash -c 'cargo clippy --all-targets --all-features --tests --benches -- -D warnings'
        language: rust
        files: \.rs$
        pass_filenames: false
      - id: cargo-test
        name: cargo test
        description: unit test for the project
        entry: bash -c 'cargo nextest run --all-features'
        language: rust
        files: \.rs$
        pass_filenames: false

[socutes]

很感谢你的建议，我需要自我检讨，因为认为是 Demo 项目所以很多细节忽略了，这些细节在主项目会做的更好一些。

因为这两天在加最后两个章节课程的代码，所以目前代码运行还有些问题，我今天会 fix 掉这个错误。

pre-commit 检查我今天也会加上

非常非常感谢你的建议～

[fan-tastic-z]

很感谢你的建议，我需要自我检讨，因为认为是 Demo 项目所以很多细节忽略了，这些细节在主项目会做的更好一些。

因为这两天在加最后两个章节课程的代码，所以目前代码运行还有些问题，我今天会 fix 掉这个错误。

pre-commit 检查我今天也会加上

非常非常感谢你的建议～

理解理解