On 1/16/26 10:30, Jamin Lin wrote:
----------------------------------------------------------------
vbootrom update pull request
----------------------------------------------------------------
Hi Jamin!
I've become curious.
You sent a pull request (and second one), listing multiple
people in To: and Cc: fields. Usually, this is done when
you're sending a patch series for review. For a pull request,
usually it's enough to list [email protected] in the To:
field, and maybe the current person who's managing the master
branch, who's merging the pull requests. Listing multiple
people for the pull request makes not much sense, since a
pullreq is something which is (almost) cast in stone already,
it is being merged as-is, there's no way to comment on it or
to modify it - since the whole pull request is being applied
to the master branch as-is.
And another question is why did you add my address to the
list of recipients? Is it for me to review the changes, or
to pick them up through qemu-trivial tree (in which case it
shouldn't be pull request), or to pick it up for qemu-stable?
Thanks,
/mjt
