[coreboot-gerrit] Change in coreboot[master]: Documentation: Revise "24 hours wait period" rules

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43484 )

Change subject: Documentation: Revise "24 hours wait period" rules
......................................................................

Patch Set 4: Code-Review+2

(3 comments)

https://review.coreboot.org/c/coreboot/+/43484/2/Documentation/getting_start... 
File Documentation/getting_started/gerrit_guidelines.md:

https://review.coreboot.org/c/coreboot/+/43484/2/Documentation/getting_start... 
PS2, Line 56: The change itself should be as limited in scope
            : and impact as possible to make it simple to assess the impact.
> Ack

I agree that mentioning particular changes isn't a good idea. I hope I can revisit this someday.

https://review.coreboot.org/c/coreboot/+/43484/2/Documentation/getting_start... 
PS2, Line 61: * Trivial changes that deal with minor issues like inconsistencies in
> I defused the entire "trivial change" exception by noting that if they're trivial enough they can al […]

Ack

https://review.coreboot.org/c/coreboot/+/43484/4/Documentation/getting_start... 
File Documentation/getting_started/gerrit_guidelines.md:

https://review.coreboot.org/c/coreboot/+/43484/4/Documentation/getting_start... 
PS4, Line 81: our code, the focus should be on fixing the issue, not on assigning blame.
+3