Re: [PATCH 1/1] [gdb]: add git trailer information on gdb/MAINTAINERS
Eli Zaretskii via Gdb-patches <[email protected]>
| Newsgroups | gmane.comp.gdb.patches,gmane.comp.gdb.devel |
|---|---|
| Message-ID | <[email protected]> |
> Date: Tue, 16 May 2023 15:40:38 -0400 > Cc: [email protected], [email protected] > From: Simon Marchi <[email protected]> > > > I don't think I'm in a position to put forward suggestions, since I'm > > not sure I have a good understanding of the process. I only use > > Approved-By when I can approve the entire patch, not just parts of it. > > But maybe I'm wrong in that. > > If this happens, I think it's fine to say "the documentation parts are > approved" and following with your Approved-By. If you want to be > extra-clear, add "but the rest needs to be approved by someone else". > The patch will end up with multiple Approved-Bys. I'd like to hear from more maintainers that this is how they see that tag. My fear is that someone mechanically scans the discussion thread for the tags, in which case human-readable qualifications will go unnoticed. All in all, I feel that this aspect of our process is not well defined.