Skip to content

Draft: Ft/commit hooks

Nyan Helsing requested to merge NyanHelsing/firefish:ft/commit-hooks into develop

What does this PR do? (Please give us a brief description of what this PR does.)

This makes sure that any PRs automatically are formatted correctly through the use of git hooks.

Adds some git hooks that will automatically run biome check--apply (but not apply-unsafe) on any staged js/ts/etc. Also adds a push hook to run format on the whole repo, so rust also will get formatted.

This uses lefthook to do this because it is very fast and more stack-agnostic than things like pre-commit or husky https://github.com/evilmartians/lefthook

Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
  • I have made sure to test this pull request

- [ ] I have made sure to run pnpm run format before submitting this pull request

If this merge request makes changes to the Firefish API, please update docs/api-change.md

  • I updated the document / This merge request doesn't include API changes

Merge request reports