tree ae17e686ee36c5caf6813a58752975ac9bb65703
parent cf928f1d78b675f1cec7039d8b345f810c8f48fe
author Edwin Kempin <ekempin@google.com> 1680691508 +0000
committer Edwin Kempin <ekempin@google.com> 1680692053 +0000

Drop "Patch Set: <num>" prefix from change message that lists owned paths

This prefix triggers special logic in the web frontend. If this prefix
is present the frontend parses approvals from the message and shows
approval chips for them. Due to this it looks like approvals have been
applied twice. If a code-owners approval is applied and the message to
inform about the owned paths is posted asynchronously there are 2 change
messages and the web frontend shows an approval chip for both of them:

1. "Patch Set X: Code-Owner+1" posted by Gerrit core
2. "Patch Set X: By voting Code-Review+1 the following files are now
   code-owner approved by YYY: ..." posted by the code-owners plugin

By dropping the "Patch Set: <num>" prefix from the second change
message, the approval chip will only be shown for the first message.

Dropping the patch set information from the second change message is
fine, since the patch set to which the message relates is still
available from the change message meta data.

This problem started happening after the code-owners plugin has been
changed to post the code-owner approved paths asynchronously in a
separate change message, rather then extending the change message that
Gerrit core posts about the approval (see change I129e611af).

Bug: Google b/276276203
Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I97a9bf0f8818abf31a071f88260359fbe677b30e