Re: [Wireshark-dev] Assigned reviewers

2021-01-06 Thread Dario Lombardo
> > >> Do 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

Re: [Wireshark-dev] Assigned reviewers

2021-01-06 Thread Pascal Quantin
Le mer. 6 janv. 2021 à 11:26, Dario Lombardo a écrit : > > > On Wed, Jan 6, 2021 at 9:38 AM Pascal Quantin > wrote: > >> Hi Jonathan, >> >> Le mer. 6 janv. 2021 à 05:39, Jonathan Nieder a >> écrit : >> >>> Hi wiresharks, >>> >>> Context: >>>

Re: [Wireshark-dev] Assigned reviewers

2021-01-06 Thread Dario Lombardo
On Wed, Jan 6, 2021 at 9:38 AM Pascal Quantin wrote: > Hi Jonathan, > > Le mer. 6 janv. 2021 à 05:39, Jonathan Nieder a > écrit : > >> Hi wiresharks, >> >> Context: >> https://gitlab.com/wireshark/wireshark/-/merge_requests/1313#note_478706594 >> >> In Gerrit times, a person could add someone

Re: [Wireshark-dev] Assigned reviewers

2021-01-06 Thread Pascal Quantin
Hi Jonathan, Le mer. 6 janv. 2021 à 05:39, Jonathan Nieder 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

[Wireshark-dev] Assigned reviewers

2021-01-05 Thread Jonathan Nieder
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