Validation of commit message

[djantea]

Hi,

This is probably related to the Roadmap point "Validation of commits".

I started to work on a mechanism to automatically validate the Git commit message by using a commit-msg git hook. It uses two mix tasks:

• mix git_ops.check_message - receives the path to a commit message and validates if it follows the Conventional Commits spec using the existing parser in the project;
• mix git_ops.msg_hook - install / uninstall a Git commit-msg hook that calls the above task. It does nothing if the hook already exists. Some inspiration drown from here: https://github.com/qgadrian/elixir_git_hooks

I have pushed these in the following branch: https://github.com/djantea/git_ops/tree/comm_msg_validation

The reasons I have not created the pull request are:

1. To have a chance to maybe get some feedback from you @zachdaniel to see if this is something that is in the line with what you were thinking
2. To write test cases for the new introduced mix tasks.

Something that could be part of a subsequent pull request:

3. To see if there is a way to automatically run mix git_ops.msg_hook when compiling the project that includes git_ops as a dependency, as a way to share the commit message validation mechanism with the team working on the project.

[zachdaniel]

This sounds very exciting! I will have some time to look at this this evening. This is a big contribution, I’m really looking forward to having this functionality.

[zachdaniel]

Sorry, I've been crazy busy lately. This isn't off my radar :D

[djantea]

I have pushed to https://github.com/djantea/git_ops/tree/comm_msg_validation the test cases for the new introduced mix tasks: git_ops.check_message and git_ops.message_hook (renamed it from mix git_ops.msg_hook).

[djantea]

Sorry, I've been crazy busy lately. This isn't off my radar :D

That is fine. Please let me know if you prefer to create the pull request and review from there, or review now from https://github.com/djantea/git_ops/tree/comm_msg_validation and create the pull request after. Thanks.

[zachdaniel]

I've looked over the bulk of the changes, but haven't had a time to do a line-by-line review. However, it looks like an awesome change. Go ahead and make the PR, and I'll do some more in-depth review and get it merged.

[djantea]

Go ahead and make the PR

Done

[asummers]

This can be closed via #25