merge request review request of a team should indicate such

[Monty Taylor]

When someone requests that a team review a merge request, the emails sent out are addressed to each individual rather than to the team. This can have the effect of misleading someone into thinking that they were explicitly requested to personally review the merge request. If they are operating solely over email, they are missing that the request as simply for any member of the team to do the review.

It would be stellar if the email could indicate that the request was made of the team, so that the reviewer knows what his status is related to the merge request just by reading the notification email.

[Aaron Bentley (abentley)]

Don't we already do that? We say "Your team bzr-core is requested to review the proposed merge of lp:~bialix/bzr/deprecate into lp:bzr." or "You are requested to review the proposed merge of lp:~jelmer/bzr/merge-dir-prober into lp:bzr."

[Monty Taylor (mordred)]

On 08/18 I got this message:

You have been requested to review the proposed merge of lp:~ewanmellor/nova/xenapi-concurrency-model into lp:nova.

Related to this merge request:

https://code.edge.launchpad.net/~ewanmellor/nova/xenapi-concurrency-model/+merge/32939

You will note that I am not on the list of people who were personally asked to review the merge request. The email was sent directly to my personal address and not to a mailing list, and I got this header:

X-Launchpad-Message-Rationale: Reviewer @nova