Skip to content

Add pre-commit#792

Closed
cclauss wants to merge 1 commit intomavlink:mainfrom
cclauss:pre-commit
Closed

Add pre-commit#792
cclauss wants to merge 1 commit intomavlink:mainfrom
cclauss:pre-commit

Conversation

@cclauss
Copy link
Copy Markdown
Contributor

@cclauss cclauss commented Sep 9, 2025

Learn more about pre-commit at https://pre-commit.com

To enable these pre-commit hooks run

  • uvx pre-commit install or
  • pipx install pre-commit or
  • brew install pre-commit, etc.

Then, in the project root directory, run pre-commit install

% pre-commit autoupdate

[https://github.com/codespell-project/codespell] already up to date!
[https://github.com/tox-dev/pyproject-fmt] already up to date!
[https://github.com/abravalheri/validate-pyproject] already up to date!

% pre-commit run --all-files

codespell................................................................Passed
pyproject-fmt........................................(no files to check)Skipped
Validate pyproject.toml..............................(no files to check)Skipped

Related to:

@julianoes
Copy link
Copy Markdown
Collaborator

I'm aware of pre-commit hooks but not a big fan. I understand their purpose but it breaks my workflow when I do a git commit -p then select the files, just to then fail the hook and lose my staged files.

I think for this repo, which doesn't attract a lot of contribution anyway, just having the checks added to CI and the README is enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants