On 5/8/19 12:59 PM, Juergen Gross wrote: > What about variant 2b: > > 1. In order to get a change to a given file committed, it must have > an Ack or Review from at least one maintainer of that file other than > the submitter. > > 2. In the case the submitter is a maintainer of a modified file it must > have an Ack or Review from either a "nested" maintainer or a Designated > reviewer of that file.
That changes the current meaning of "Designated Reviewer" somewhat. At the moment, it only means "Send this person an email"; if we did that, it would mean, "This person is more trusted to do reviews than others". Not sure what I think about that yet. >> + The meaning of nesting >> + ====================== > > Everywhere else tabs are used for indenting. I did try to get my editor to use tabs. I'll try harder next time I guess. :-/ -George _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
