Attention is currently required from: Paul Menzel, Stefan Reinauer, Jan Dabros. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52265 )
Change subject: Documentation/gerrit: Document our Gerrit user roles and procedures ......................................................................
Patch Set 5: Code-Review+1
(2 comments)
File Documentation/getting_started/gerrit_guidelines.md:
https://review.coreboot.org/c/coreboot/+/52265/comment/c5ae85ae_a8437912 PS2, Line 347: by themselves or others, : at the regular [coreboot leadership meetings](../community/forums.md) : where a decision is made.
The idea behind this is to provide some checks & balances, otherwise we might at some point have bla […]
I think it's a reasonable approach. It's not like the core developer list gets updated too often. Plus, considering that the privileges of the core developers group can easily be abused, having a general consensus and a record (meeting minutes) for new additions to this group is a good idea.
File Documentation/getting_started/gerrit_guidelines.md:
https://review.coreboot.org/c/coreboot/+/52265/comment/738dde6f_b1e78c52 PS5, Line 329: -1 I thought registered users could also -1 changes?