On 06/05/19 03:10, Ni, Ray wrote: > Hi everyone, > > Hao pushed the patch because: > 1. it's a bug fix > 2. it got a R-b. > I don't think it's his fault. > > For #1, it's gap in process. > For #2, it's my fault. Because even the patch title says EmulatorPkg but the > patch itself changes MdeModulePkg, I am still the person who can approve > changes in both packages. > > In Maintainers.txt: > R: Ray Ni <ray...@intel.com> > (especially for Bus, Universal/Console, Universal/Disk, > Universal/BdsDxe and related libraries, header files) > > The UiApp actually is not in the "especially" part, but the word "especially" > means I can also R-b to other components. In fact, I am not quite qualified on > the other components, e.g.: UiApp. > So I propose to assign clear reviewers for each components in MdeModulePkg. > What do you think?
Fully agreed. MdeModulePkg is huge and we should have fine-grained maintainership assignments. Thanks Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#41925): https://edk2.groups.io/g/devel/message/41925 Mute This Topic: https://groups.io/mt/31907735/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
