Page MenuHomeFreeBSD

Suggest commit title in prepare-commit-msg hook
AbandonedPublic

Authored by grembo on May 7 2021, 11:43 AM.
Tags
None
Referenced Files
F97153684: D30158.diff
Fri, Sep 27, 11:21 PM
Unknown Object (File)
Mon, Sep 23, 4:21 AM
Unknown Object (File)
Sun, Sep 22, 4:40 PM
Unknown Object (File)
Thu, Sep 19, 7:40 AM
Unknown Object (File)
Tue, Sep 17, 2:58 AM
Unknown Object (File)
Sat, Sep 7, 9:20 PM
Unknown Object (File)
Sat, Aug 31, 12:02 PM
Unknown Object (File)
Sat, Aug 31, 7:11 AM
Subscribers

Details

Reviewers
None
Group Reviewers
portmgr
Summary

This is done based on the files affected by the commit. Suggests up
to two ports at a time and goes to "many" from there.

Unintrusive, yet helpful to get the commit title right (forgetting
the convention, typos, or committing a port by accident).

IMHO better than creating a draconian commit hook, nudging
committers to do the right thing by making their lives easier.

This is what I use personally, I was asked in private if there
was a chance to get this into the repo, so I wanted to try at
least. No hard feelings if you feel like this is unnecessary.

Test Plan
echo something >>devel/ice/Makefile
echo somethingelse >>editors/joe/Makefile
git commit -a

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 39072
Build 35961: arc lint + arc unit

Event Timeline

Prevent extra empty line when ammending.

Add quoting like in the original template.

Also bring in .arcconfig change from D30162.

Sorry for all the noise, .arcconfig vs. git migration.

@adridg As you've been sending out regular messages about following the commit message conventions. What do you think?

Works for me (with a sample size n=1), having the category/port: already filled in is nice. It demonstrates nicely what is possible.

@mat Next time, please show some basic respect and comment on a review that has been made obsolete by your own contribution (especially since I made it clear that I'm ok if you don't like it).