You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Big thanks to @mxr for the pull requests, this release incorporate's @mxr's changes, removing my workaround instructions, and tweaking when we load the Afin library.
Drop unnecessary args from .pre-commit-hooks.yaml, README fixes.
- Drop the args from .pre-commit-hooks, we don't need them
- Remove extra whitespace from README
In using this commit-msg hook script more, I've found that most common commit messages are about one line of text (120 characters or so) and setting the default MIN_COMMIT_MSG_LENGTH value for the commit-msg-sentiment.py script to 120 results in far fewer false warnings. This value is adjustable via an environment variable. I will document that in the README soon.
It turns out that the Pre-commit framework is somewhat file-oriented. In order to ensure that the commit-msg-sentiment.py hook script runs for every commit, we have to tell Pre-commit to do so. Oops.