Page MenuHomeFreeBSD

committers-guide: update commit trailer descriptions
ClosedPublic

Authored by emaste on Feb 23 2021, 6:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 28, 9:53 PM
Unknown Object (File)
Mon, Oct 28, 9:52 PM
Unknown Object (File)
Mon, Oct 28, 9:51 PM
Unknown Object (File)
Mon, Oct 28, 9:33 PM
Unknown Object (File)
Tue, Oct 22, 8:09 AM
Unknown Object (File)
Mon, Oct 21, 12:57 AM
Unknown Object (File)
Mon, Oct 21, 12:56 AM
Unknown Object (File)
Mon, Oct 21, 12:56 AM
Subscribers

Details

Summary
  • Add Reported by, Tested by, Fixes, MFH.
  • Mention that Submitted by is typically not used with git.
  • Add more cases to Approved by.
  • Move Sponsored by: to the same place it appears in the template.

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste created this revision.

Just one small thing, and as long as it builds (which it should, from the looks of it), it looks good to me.

documentation/content/en/articles/committers-guide/_index.adoc
1649–1650

I think there should be some mention of the rules for this field, as I'm not entirely sure they're documented elsewhere, and it seems like a good place for the canonical definition, ie:

Hash length (lwhsu in D28881 mentioned git rev-parse --short for generating the shortest identifiable hash), and how titles can either be truncated or pushed onto a new line.

I forgot to accept this, didn't I.

I know you'll fix these before you push it, though. :)

This revision is now accepted and ready to land.Feb 24 2021, 8:43 AM

and as long as it builds

As an aside, checking this is so much nicer with asciidoc/hugo.