Derek Chen-Becker <[email protected]> writes:

> Patches attached for the numeric priorities as well as the removal of the
> "show" parameter and an updated `mk/targets.mk' to install and remove the
> git hook scripts.

Thanks! I have several comments on the hooks.
1. The hooks still have # This file is not part of GNU Emacs
   Those lines should be removed. The file headers should also be
   modified to refer to Org mode. GNU Emacs -> Org mode.
2. If you look at the rest of CONTRIBUTE.org, you will see that it
   refers to WORG for basically everything. We should probably modify
   WORG page and refer to it instead. (At some point, that page should
   go back to the main repo, but that's a different story)
3. You should document how to disable the hooks for people who do not
   want them.
4. You should document how to remove the installed hooks (make cleanall)

-- 
Ihor Radchenko // yantar92,
Org mode maintainer,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>

Reply via email to