CONTRIBUTING.md: Add line about commit messages

[cdayjr]

• Have you followed the guidelines for contributing?
• [N/A] Have you checked that there aren't other open pull requests for the same formula update/change?
• [N/A] Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• [N/A] Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• [N/A] Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

---

I saw there was a bot check for the commit messages after I had already went through the checklist above and got my PR ready, I figure it'd make sense to add a line about the commit styles in the CONTRIBUTING.md guide.

[carlocab]

Thanks, @cdayjr. One small suggestion.

[carlocab]

Thanks @cdayjr!

[carlocab]

Will leave this open for a bit longer to get more eyes on it.

[nandahkrishna]

This looks good to me. Although there are some instructions right at the bottom (the point with run git commit ...), having this at the top with a link to the Formula Cookbook section would definitely make it more noticeable and should hopefully translate into more contributors adhering to the guidelines.

Just a small suggestion (I don't feel too strongly about this): We may want to prefix the first sentence with If you are creating a pull request, . I say this because right below the message, we have the To report a bug section, and that doesn't require committing.

[nandahkrishna]

Thanks!

[MikeMcQuaid]

This is mentioned in "To add a new formula" below and brew bump-formula-pr will handle it for you too. Thoughts?

[carlocab]

I still pull out my "please use homebrew/core style in your commit messages" saved reply at least once a day, so I think a lot of users still miss it.

I'm fine with making this less verbose (i.e. back to @cdayjr's original suggestion 😅) so easier to scan.

[MikeMcQuaid]

Cool, I don't feel strongly, I'll leave to another maintainer to merge this.

[carlocab]

@cdayjr I think I'm happy to go back to your original suggestion to keep things less verbose. Verbosity discourages contributors. Or at least, discourages contributors from reading the guidelines. Sorry for flip-flopping on this!

Thoughts, @nandahkrishna?

[nandahkrishna]

I'm fine with the original one as well.

[cdayjr]

Thanks all. I have removed the additional commits from this branch.

[carlocab]

Thanks, @cdayjr!