On Tue, Nov 06, 2018 at 03:31:08PM +0000, Peter Maydell wrote: > On 6 November 2018 at 15:15, Philippe Mathieu-Daudé <phi...@redhat.com> wrote: > > No worry, I'm not mad at all, but there might be an issue in your git PR > > workflow, this series also missed your maintainer S-o-b. > > > > Peter: Can you add a such check in your scripts? (during next merge window, > > no hurry). > > > > Rather than your scripts, this should be in scripts a maintainer can run > > locally, such ./scripts/checkpatch.pl --maintainer or > > ./scripts/checkseries.xx. > > I have such a check in my own make-pullreq script which I use > for arm pull requests: > https://git.linaro.org/people/peter.maydell/misc-scripts.git/tree/make-pullreq#n98 > > I can't do much about other submaintainers' workflows > (and the block tree's "sub-submaintainers" setup means > "check all commits have a signoff from the same person as > the merge commit did" won't work either.) > > thanks > -- PMM
I don't have anything validating that but I agree it's a good idea. Will add, thanks!
