Skip to content

Add githooks from alpine

Pablo Correa Gomez requested to merge githooks into master

Get the githooks from alpine, with the following changes:

  • Make it possible to parse the additionally nested "device" folder with the exceptions mentioned in our COMMITSTYLE
  • Ignore "device/*" as a possible prefix for the commit
  • Replace abuild references with pmbootstrap

Relates #2055 (closed)

I have on purpose not implemented most of the ideas mentioned by @calebccff. Many of those seem to be complex checks (some houndred lines of python code in CI), and are already done in CI. Another reason is the dependencies that might be required. If you think there's a sensible, fast, and simple way to implement some of those, I'd be open for it. But I believe things like shellcheck, or checking pkgver changes are out of the scope of the hooks.

Instead, I've only included the very basic quality of life improvements from alpine, adapted to pmOS COMMITSTYLE. This might seem small, but my experience in aports is that they feel pretty nice and save quite some time.

Merge request reports