Wireshark mailing list archives
Re: Assigned reviewers
From: Pascal Quantin <pascal () wireshark org>
Date: Wed, 6 Jan 2021 12:04:22 +0100
Le mer. 6 janv. 2021 à 11:26, Dario Lombardo <lomato () gmail com> a écrit :
On Wed, Jan 6, 2021 at 9:38 AM Pascal Quantin <pascal () wireshark org> wrote:Hi Jonathan, Le mer. 6 janv. 2021 à 05:39, Jonathan Nieder <jrnieder () gmail com> a écrit :Hi wiresharks, Context: https://gitlab.com/wireshark/wireshark/-/merge_requests/1313#note_478706594 In Gerrit times, a person could add someone as a reviewer to a change to request review, the reviewer could remove themselves if they were unavailable, and so on. What is the equivalent in the GitLab world? More concretely: - when a change is ready to review, how do I say so?All opened threads are resolved and the submitter can add a comment to ping us. A reviewer can be explicitly added in the right column of the Gitlab GUIDo you mean assignee? I guess so, but I'd like to clear it, since the reviewer and assignee were separate in Gerrit.
No I really meant reviewer as I was considering the assignee as the person that will ultimately schedule the merge. You can have more than one reviewer. But I'm open to any workflow we might define.
- if a review seems to be stalled, what's the best place to poke?Writing a comment in the MR; we are almost all volunteers doing this on our spare time so sometimes real life collides and a given change can get out of the radar - if I would like to review a change, how should I signal interest?Everybody is free to put comments in a MR - what happens when a change has been approved and it is time to mergeit? Where can I read about the bot that does that?One of the core developer approves the change and schedules it for mergeThe Core devels are able to rebase and merge a MR. However the race for merge with other MRs could make the merge harder. That's why we can assign the MR to the bot that automatically rebases the change until the merge actually happens. But it's not a must.
As only core developers can do this operation, that's what I meant when saying "schedule for merge" but did not enter into the details. Thanks for clarifying it.
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Assigned reviewers Jonathan Nieder (Jan 05)
- Re: Assigned reviewers Pascal Quantin (Jan 06)
- Re: Assigned reviewers Dario Lombardo (Jan 06)
- Re: Assigned reviewers Pascal Quantin (Jan 06)
- Re: Assigned reviewers Dario Lombardo (Jan 06)
- Re: Assigned reviewers Dario Lombardo (Jan 06)
- Re: Assigned reviewers Pascal Quantin (Jan 06)