Page MenuHomeFreeBSD

Harmonize the cross repo advice.
ClosedPublic

Authored by imp on Jul 2 2026, 4:36 PM.
Tags
None
Referenced Files
F166888592: D58007.id182009.diff
Mon, Aug 17, 1:25 PM
F166885589: D58007.id181226.diff
Mon, Aug 17, 12:54 PM
F166861612: D58007.id.diff
Mon, Aug 17, 6:05 AM
Unknown Object (File)
Sun, Aug 16, 4:32 PM
Unknown Object (File)
Tue, Aug 11, 7:41 PM
Unknown Object (File)
Mon, Aug 10, 10:13 AM
Unknown Object (File)
Sun, Aug 9, 3:55 PM
Unknown Object (File)
Sat, Aug 8, 3:27 PM
Subscribers
None

Details

Summary

The other committer's guide was written years before the cross repo
commit bit stuff. Harmonize and expand a little the advice given to make
it also match the tone of the new policy and our usual practices being a
little more relaxed than what was documented previously.

Sponsored by: Netflix

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 74484
Build 71367: arc lint + arc unit

Event Timeline

imp requested review of this revision.Jul 2 2026, 4:36 PM
imp created this revision.
documentation/content/en/articles/committers-guide/_index.adoc
272

I wonder if we want to relax this to "All committers" vs just Doc? I'm fine with keeping the existing language though.

2645

I really want this case to just be "Reviewed by;" and not "Approved by:". That is, the point is to get review of the change itself not permission to commit a change separate from reviewing it.

This revision was not accepted when it landed; it landed in state Needs Review.Jul 14 2026, 11:01 PM
This revision was automatically updated to reflect the committed changes.