prepare-commit-msg: add 50 column marker
ClosedPublic
Actions

Authored by emaste on Jun 9 2024, 5:27 PM.

Tags

None

Referenced Files

	F166267543: D45538.id139662.diff
	Wed, Aug 12, 3:23 PM

	F166207876: D45538.id.diff
	Wed, Aug 12, 9:08 AM

	Unknown Object (File)
	Tue, Aug 11, 2:45 PM

	Unknown Object (File)
	Sat, Aug 8, 7:45 PM

	Unknown Object (File)
	Fri, Aug 7, 7:11 PM

	Unknown Object (File)
	Fri, Aug 7, 5:21 PM

	Unknown Object (File)
	Thu, Aug 6, 9:16 AM

	Unknown Object (File)
	Thu, Aug 6, 3:52 AM

View All Files

Subscribers

lwhsu

Details

Reviewers

imp
lwhsu

Commits

rG87e5b17c149e: prepare-commit-msg: add 50 column marker

Summary

A common convention suggests a 50 column commit subject line (the first
line of the commit message). Add a 50 column marker to the commit
message template

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste requested review of this revision.Jun 9 2024, 5:27 PM

emaste created this revision.

How about adding a second comment line says that the 50 is for subject and 72 is for body?

This revision is now accepted and ready to land.Jun 9 2024, 10:13 PM

How about adding a second comment line says that the 50 is for subject and 72 is for body?

I started writing some extra text but then it seems we should also comment on the blank 2nd line and it got to be a tutorial rather than a quick reference. Perhaps we could add a link to the section in the committer's guide. For now I'll commit this bit.

Closed by commit rG87e5b17c149e: prepare-commit-msg: add 50 column marker (authored by emaste). · Explain WhyJun 10 2024, 2:49 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rG87e5b17c149e: prepare-commit-msg: add 50 column marker.