[SUMMARY] Re: [PATCH v2 0/2] This is the pull request title
developer via Claudio Bantaloukas
claudio.bantaloukas+developer@arm.com
Tue Apr 14 05:37:10 GMT 2026
This is a summary of discussions relative to the merge request created by (developer) <developer+claudio.bantaloukas@arm.com> titled
This is the pull request title
since its creation.
Description: Here is a description of the pull request.
This is a demo to show how a patch is created, then how emails are sent:
- when a review is requested
- when someone makes a free form comment
- when someone comments on the code using the review mechanism
- when someone approves or requests changes
- when the pull request is closed (as, for now, we can't merge using the UI)
The full and up to date discussion can be found at http://localhost:3000/fadmin/d/pulls/14
The merge request has been closed without being merged directly on the forge repository.
On 2026-04-14 05:23:18+00:00, (developer) <developer+claudio.bantaloukas@arm.com> requested that (fadmin) <fadmin@arm.com> review the code:
On 2026-04-14 05:24:19+00:00, (developer) wrote:
Here is a free form comment.
I have requested code review from two users.
On 2026-04-14 05:27:00+00:00, (fadmin) wrote:
<!-- bmm:send -->
Here is a comment by fadmin that should be sent as an email because it contains <!-- bmm:send -->.
On 2026-04-14 05:30:50+00:00, (reviewer) <reviewer@arm.com> commented on the code:
I will be happy to review.
This is a single comment review. It will trigger a patch series email sending because it's a review on a changed branch.
On 2026-04-14 05:34:40+00:00, (fadmin) wrote:
Comments by the bot users do not show up in the emails unless they contain
<!-- bmm:send -->
On 2026-04-14 05:35:30+00:00, (reviewer) <reviewer@arm.com> requested changes to the code:
I think some changes are necessary before I can approve this patch.
> +++ example-file
> @@ -2,1 +2,4 @@
>
> +Hello world.
> +
> +This is the second change, it will trigger a patch series posting.
I think this change could be improved
> +++ example-file
> @@ -3,3 +6,4 @@
> +
> Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.
>
> Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
We could perhaps trim this bit and it lacks a newline.
On 2026-04-14 05:36:51+00:00, (reviewer) <reviewer@arm.com> approved the changes:
Actually I have changed my mind. Yay Latin!
On 2026-04-14 05:37:07+00:00, (reviewer) wrote:
It is now time to close and merge manually.
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
More information about the Test-list
mailing list